[RFC,v1,2/4] fff-hoods: set hood to none if on station mode

Submitted by Tim Niemeyer on April 3, 2018, 7:27 p.m.

Details

Message ID 20180403192749.28637-3-tim@tn-x.org
State Superseded
Headers show

Commit Message

Tim Niemeyer April 3, 2018, 7:27 p.m.
Signed-off-by: Tim Niemeyer <tim@tn-x.org>
---

 src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 3 +++
 1 file changed, 3 insertions(+)

Patch hide | download patch | download mbox

diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
index 1b9fa2b..a825eea 100755
--- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
+++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
@@ -98,6 +98,9 @@  else
 			rm -f "$hoodfilecopy" # delete this, so interfaces are recreated if reconnect with unchanged hood file takes place
 			rm -f "$sectorcopy" # always delete: no broadcast for isolated device
 			rm -f "$sectortmp"
+
+			uci -q set "system.@system[0].hood="
+			uci -q commit system
 		
 			sleep 30 # Wait for the config AP, which may be created at the same time as this script has started
 

Comments

Christian Dresel June 10, 2018, 6:28 a.m.
hi

sieht schlüssig aus, bei kurzen Tests hatte ich auch keine Probleme das
der Nodewatcher oder das Webui deshalb hängen bleibt oder Unsinn macht
(was ich zuerst befürchtet habe). Sonst wird es glaub ich nirgens mehr
ausgewertet.

Reviewed-by: Christian Dresel <fff@chrisi01.de>

mfg

Christian


On 03.04.2018 21:27, Tim Niemeyer wrote:
> Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> ---
> 
>  src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> index 1b9fa2b..a825eea 100755
> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> @@ -98,6 +98,9 @@ else
>  			rm -f "$hoodfilecopy" # delete this, so interfaces are recreated if reconnect with unchanged hood file takes place
>  			rm -f "$sectorcopy" # always delete: no broadcast for isolated device
>  			rm -f "$sectortmp"
> +
> +			uci -q set "system.@system[0].hood="
> +			uci -q commit system
>  		
>  			sleep 30 # Wait for the config AP, which may be created at the same time as this script has started
>  
>
Adrian Schmutzler June 10, 2018, 10:27 a.m.
Hallo,

ich hätte an der Stelle gefühlt ein uci del gemacht. Ist im Endeffekt wahrscheinlich wurscht, aber spricht da was dagegen?

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
> Of Christian Dresel
> Sent: Sonntag, 10. Juni 2018 08:28
> To: Tim Niemeyer <tim@tn-x.org>; franken-dev@freifunk.net
> Subject: Re: [RFC PATCH v1 2/4] fff-hoods: set hood to none if on station
> mode
> 
> hi
> 
> sieht schlüssig aus, bei kurzen Tests hatte ich auch keine Probleme das der
> Nodewatcher oder das Webui deshalb hängen bleibt oder Unsinn macht
> (was ich zuerst befürchtet habe). Sonst wird es glaub ich nirgens mehr
> ausgewertet.
> 
> Reviewed-by: Christian Dresel <fff@chrisi01.de>
> 
> mfg
> 
> Christian
> 
> 
> On 03.04.2018 21:27, Tim Niemeyer wrote:
> > Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> > ---
> >
> >  src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > index 1b9fa2b..a825eea 100755
> > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > @@ -98,6 +98,9 @@ else
> >  			rm -f "$hoodfilecopy" # delete this, so interfaces are
> recreated if reconnect with unchanged hood file takes place
> >  			rm -f "$sectorcopy" # always delete: no broadcast for
> isolated device
> >  			rm -f "$sectortmp"
> > +
> > +			uci -q set "system.@system[0].hood="
> > +			uci -q commit system
> >
> >  			sleep 30 # Wait for the config AP, which may be
> created at the
> > same time as this script has started
> >
> >
Tim Niemeyer June 10, 2018, 10:31 a.m.
Am Sonntag, den 10.06.2018, 12:27 +0200 schrieb
mail@adrianschmutzler.de:
> Hallo,
> 
> ich hätte an der Stelle gefühlt ein uci del gemacht. Ist im Endeffekt
> wahrscheinlich wurscht, aber spricht da was dagegen?
Hatte sich Fabian schon gewünscht. Habe ich geprüft und bereits
eingebaut.

Tim

> Grüße
> 
> Adrian
> 
> > -----Original Message-----
> > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On
> > Behalf
> > Of Christian Dresel
> > Sent: Sonntag, 10. Juni 2018 08:28
> > To: Tim Niemeyer <tim@tn-x.org>; franken-dev@freifunk.net
> > Subject: Re: [RFC PATCH v1 2/4] fff-hoods: set hood to none if on
> > station
> > mode
> > 
> > hi
> > 
> > sieht schlüssig aus, bei kurzen Tests hatte ich auch keine Probleme
> > das der
> > Nodewatcher oder das Webui deshalb hängen bleibt oder Unsinn macht
> > (was ich zuerst befürchtet habe). Sonst wird es glaub ich nirgens
> > mehr
> > ausgewertet.
> > 
> > Reviewed-by: Christian Dresel <fff@chrisi01.de>
> > 
> > mfg
> > 
> > Christian
> > 
> > 
> > On 03.04.2018 21:27, Tim Niemeyer wrote:
> > > Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> > > ---
> > > 
> > >  src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 3 +++
> > >  1 file changed, 3 insertions(+)
> > > 
> > > diff --git a/src/packages/fff/fff-
> > > hoods/files/usr/sbin/configurehood
> > > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > > index 1b9fa2b..a825eea 100755
> > > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > > @@ -98,6 +98,9 @@ else
> > >  			rm -f "$hoodfilecopy" # delete this, so
> > > interfaces are
> > 
> > recreated if reconnect with unchanged hood file takes place
> > >  			rm -f "$sectorcopy" # always delete: no
> > > broadcast for
> > 
> > isolated device
> > >  			rm -f "$sectortmp"
> > > +
> > > +			uci -q set "system.@system[0].hood="
> > > +			uci -q commit system
> > > 
> > >  			sleep 30 # Wait for the config AP, which
> > > may be
> > 
> > created at the
> > > same time as this script has started
> > > 
> > > 
> 
>