| Message ID | 20171103200701.9647-2-tim@tn-x.org |
|---|---|
| State | Accepted |
| Headers | show |
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.@system[0].latitude)
Wenn dir das so lieber ist. Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Tim Niemeyer > Sent: Freitag, 3. November 2017 21:07 > To: franken-dev@freifunk.net > Subject: [PATCH 2/2] fff-hoods: only delete hiddenapfile if ap was configured > > Signed-off-by: Tim Niemeyer <tim@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.@system[0].latitude) > -- > 2.11.0 > > -- > franken-dev mailing list > franken-dev@freifunk.net > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Am Freitag, den 03.11.2017, 21:10 +0100 schrieb mail@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@adrianschmutzler.de> > > > -----Original Message----- > > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On > > Behalf > > Of Tim Niemeyer > > Sent: Freitag, 3. November 2017 21:07 > > To: franken-dev@freifunk.net > > Subject: [PATCH 2/2] fff-hoods: only delete hiddenapfile if ap was > > configured > > > > Signed-off-by: Tim Niemeyer <tim@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.@system[0].latitude) > > -- > > 2.11.0 > > > > -- > > franken-dev mailing list > > franken-dev@freifunk.net > > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net > >
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@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@adrianschmutzler.de> > > > > > -----Original Message----- > > > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On > > > Behalf > > > Of Tim Niemeyer > > > Sent: Freitag, 3. November 2017 21:07 > > > To: franken-dev@freifunk.net > > > Subject: [PATCH 2/2] fff-hoods: only delete hiddenapfile if ap > > > was > > > > configured > > > > > > Signed-off-by: Tim Niemeyer <tim@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.@system[0].latitude) > > > -- > > > 2.11.0 > > > > > > -- > > > franken-dev mailing list > > > franken-dev@freifunk.net > > > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.n > > > et
Signed-off-by: Tim Niemeyer <tim@tn-x.org> --- src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)