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

[Tim Niemeyer]

Hi

Und applied.

Tim

Am Freitag, den 03.11.2017, 21:14 +0100 schrieb 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.n
> > > et
-------------- 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/6739340d/attachment.sig>