[PATCH 2/2] fff-hoods: only delete hiddenapfile if ap was configured

[Tim Niemeyer]

Am Freitag, den 03.11.2017, 21:10 +0100 schrieb
mail at adrianschmutzler.de:
> Wenn dir das so lieber ist.
Ja, eigentlich schon,  weil die Änderung kleiner ist.

Ich hab das aber gebaut, und wurde danach erst darauf aufmerksam, dass
es da noch ein Patch gab, wo Unklarheiten waren. Das war der "Remove
gateway-up condition for hidden AP" von dir.

Wir haben jetzt zwei Lösungen zur Auswahl, wobei ich tatsächlich meinen
favorisiere, weil er klarer ist. Dein Patch macht ja noch mehr, das
kann man ggfs separat nochmal diskutieren und danach ebenfalls applien.

Tim


> Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> 
> > -----Original Message-----
> > From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On
> > Behalf
> > Of Tim Niemeyer
> > Sent: Freitag, 3. November 2017 21:07
> > To: franken-dev at freifunk.net
> > Subject: [PATCH 2/2] fff-hoods: only delete hiddenapfile if ap was
> 
> configured
> > 
> > Signed-off-by: Tim Niemeyer <tim at tn-x.org>
> > ---
> > 
> >  src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/src/packages/fff/fff-
> > hoods/files/usr/sbin/configurehood
> > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > index 1e2831d..b10d210 100755
> > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > @@ -57,8 +57,9 @@ if [ -f "$hiddenapfile" ]; then
> >  		uci commit network
> > 
> >  		wifi
> > +
> > +		rm "$hiddenapfile"
> >  	fi
> > -	rm "$hiddenapfile"
> >  fi
> > 
> >  lat=$(uci -q get system. at system[0].latitude)
> > --
> > 2.11.0
> > 
> > --
> > franken-dev mailing list
> > franken-dev at freifunk.net
> > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
> 
> 
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 488 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20171103/03c0c998/attachment.sig>