Message ID | 20180102122044.6660-1-fff@chrisi01.de |
---|---|
State | Rejected |
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 2b9c47e..02d9a90 100755 --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood @@ -1,5 +1,10 @@ #!/bin/sh +if [ $(uci -q get fff.system.hoodfix) ]; then + echo "Hood is fix, we do nothing" + exit 1 +fi + . /usr/share/libubox/jshn.sh . /lib/functions/fff/keyxchange . /lib/functions/fff/network
Hallo Christian, habe kein Problem mit der Idee an sich, würde das aber ein bisschen anders machen: Langfristig werden wir wohl ein manuelle Hoodselektion einbauen. Deshalb und wegen der Ordnung würde ich gerne einen eigenen Block "hood" verwenden. Die Kommandos wären dementsprechend: uci set fff.hood=fff uci set fff.hood.fixed=1 uci commit fff to revert this use: uci del fff.hood.fixed ODER uci set fff.hood.fixed=0 uci commit fff Im Code dann dementsprechend: if [ "$(uci -q get fff.hood.fixed)" = "1" ]; then Später könnte man dann z.B. fff.hood.name oder fff.hood.setname nutzen, um die Hood per Namen festzulegen. Dann spammen wir den System Block nicht zu. Wenn ich es richtig verstanden habe, deaktiviert du das configurehood mit deiner Lösung komplett? Bin am überlegen, ob man das statt "fix/fixed" dann auf "disabled" umbenennen sollte? Und noch die obligatorische Klugscheißer-Anmerkung: Ich würde aus sprachlichen Gründen überall "fixed" statt "fix" verwenden. Grüße Adrian > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Christian Dresel > Sent: Dienstag, 2. Januar 2018 13:21 > To: franken-dev@freifunk.net > Subject: [PATCH] Add option to set Hoodconfiguration fix > > To set Hoodconfiguration fix use: > uci set fff.system.hoodfix=1 > uci commit > to revert this use: > uci del fff.system.hoodfix > uci commit > > Signed-off-by: Christian Dresel <fff@chrisi01.de> > --- > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 2b9c47e..02d9a90 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -1,5 +1,10 @@ > #!/bin/sh > > +if [ $(uci -q get fff.system.hoodfix) ]; then > + echo "Hood is fix, we do nothing" > + exit 1 > +fi > + > . /usr/share/libubox/jshn.sh > . /lib/functions/fff/keyxchange > . /lib/functions/fff/network > -- > 2.11.0 > > -- > franken-dev mailing list > franken-dev@freifunk.net > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Hi Christian, warum über uci configurehood disablen? Soll da später mal ein Häkchen im UI entstehen? Wenn man das bei einem Uplinkrouter macht, kommt vpn-select nie dran und es gibt keine Tunnel. Ich verstehe es nicht ganz. Am 02.01.2018 um 13:20 schrieb Christian Dresel: > To set Hoodconfiguration fix use: > uci set fff.system.hoodfix=1 > uci commit > to revert this use: > uci del fff.system.hoodfix > uci commit > > Signed-off-by: Christian Dresel <fff@chrisi01.de> > --- > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 2b9c47e..02d9a90 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -1,5 +1,10 @@ > #!/bin/sh > > +if [ $(uci -q get fff.system.hoodfix) ]; then > + echo "Hood is fix, we do nothing" > + exit 1 Bei einem gewünschtem Ausstieg ohne Fehler wäre errorlevel 0 schöner. Robert > +fi > + > . /usr/share/libubox/jshn.sh > . /lib/functions/fff/keyxchange > . /lib/functions/fff/network
hi On 02.01.2018 14:48, robert wrote: > Hi Christian, > > warum über uci configurehood disablen? Soll da später mal ein Häkchen im > UI entstehen? Wenn man das bei einem Uplinkrouter macht, kommt kann man machen, muss man nicht. Der Grund ist recht einfach: Ich hab keinen Bock das in meinen dezentralen Hoods durch $irgendwas die Hood gewechselt wird. Die Router hab ich unter meine fittiche und die sollen da bleiben wo ICH sie haben will. > vpn-select nie dran und es gibt keine Tunnel. Ich verstehe es nicht ganz. Da macht es auch keinen Sinn die Option zu setzen, mir reicht SSH wenn jemand ein WebUI machen will, sollte man das aber unbedingt in eine advanced Menü verpacken wo auch ne dicke Warnung dran steht, was schief laufen kann. mfg Christian > > > > Am 02.01.2018 um 13:20 schrieb Christian Dresel: >> To set Hoodconfiguration fix use: >> uci set fff.system.hoodfix=1 >> uci commit >> to revert this use: >> uci del fff.system.hoodfix >> uci commit >> >> Signed-off-by: Christian Dresel <fff@chrisi01.de> >> --- >> src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >> index 2b9c47e..02d9a90 100755 >> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >> @@ -1,5 +1,10 @@ >> #!/bin/sh >> >> +if [ $(uci -q get fff.system.hoodfix) ]; then >> + echo "Hood is fix, we do nothing" >> + exit 1 > Bei einem gewünschtem Ausstieg ohne Fehler wäre errorlevel 0 schöner. > > Robert >> +fi >> + >> . /usr/share/libubox/jshn.sh >> . /lib/functions/fff/keyxchange >> . /lib/functions/fff/network >
Am 02.01.2018 um 14:51 schrieb Christian Dresel: > hi > > On 02.01.2018 14:48, robert wrote: >> Hi Christian, >> >> warum über uci configurehood disablen? Soll da später mal ein Häkchen im >> UI entstehen? Wenn man das bei einem Uplinkrouter macht, kommt > kann man machen, muss man nicht. Der Grund ist recht einfach: > Ich hab keinen Bock das in meinen dezentralen Hoods durch $irgendwas die > Hood gewechselt wird. Die Router hab ich unter meine fittiche und die > sollen da bleiben wo ICH sie haben will. Das ist dann eher was fuer Leute die dran rum basteln und verstehen, was configurehood macht. Dann brauchts das nicht in der Firmware. Und wenn du so und so an die Konsole gehst, dann schalt doch einfach den cronjob aus. Dann ist Ruhe ;) Gruesse Robert > >> vpn-select nie dran und es gibt keine Tunnel. Ich verstehe es nicht ganz. > Da macht es auch keinen Sinn die Option zu setzen, mir reicht SSH wenn > jemand ein WebUI machen will, sollte man das aber unbedingt in eine > advanced Menü verpacken wo auch ne dicke Warnung dran steht, was schief > laufen kann. > mfg > > Christian > >> >> >> Am 02.01.2018 um 13:20 schrieb Christian Dresel: >>> To set Hoodconfiguration fix use: >>> uci set fff.system.hoodfix=1 >>> uci commit >>> to revert this use: >>> uci del fff.system.hoodfix >>> uci commit >>> >>> Signed-off-by: Christian Dresel <fff@chrisi01.de> >>> --- >>> src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 5 +++++ >>> 1 file changed, 5 insertions(+) >>> >>> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >>> index 2b9c47e..02d9a90 100755 >>> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >>> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >>> @@ -1,5 +1,10 @@ >>> #!/bin/sh >>> >>> +if [ $(uci -q get fff.system.hoodfix) ]; then >>> + echo "Hood is fix, we do nothing" >>> + exit 1 >> Bei einem gewünschtem Ausstieg ohne Fehler wäre errorlevel 0 schöner. >> >> Robert >>> +fi >>> + >>> . /usr/share/libubox/jshn.sh >>> . /lib/functions/fff/keyxchange >>> . /lib/functions/fff/network
hi haben jetzt ne neue Lösung die ich deutlich besser finde. Patch ist bereits auf der ML und dieses Patch kann komplett verworfen werden. mfg Christian On 02.01.2018 15:00, robert wrote: > > > Am 02.01.2018 um 14:51 schrieb Christian Dresel: >> hi >> >> On 02.01.2018 14:48, robert wrote: >>> Hi Christian, >>> >>> warum über uci configurehood disablen? Soll da später mal ein Häkchen im >>> UI entstehen? Wenn man das bei einem Uplinkrouter macht, kommt >> kann man machen, muss man nicht. Der Grund ist recht einfach: >> Ich hab keinen Bock das in meinen dezentralen Hoods durch $irgendwas die >> Hood gewechselt wird. Die Router hab ich unter meine fittiche und die >> sollen da bleiben wo ICH sie haben will. > Das ist dann eher was fuer Leute die dran rum basteln und verstehen, was > configurehood macht. Dann brauchts das nicht in der Firmware. > Und wenn du so und so an die Konsole gehst, dann schalt doch einfach den > cronjob aus. Dann ist Ruhe ;) > > Gruesse > Robert >> >>> vpn-select nie dran und es gibt keine Tunnel. Ich verstehe es nicht ganz. >> Da macht es auch keinen Sinn die Option zu setzen, mir reicht SSH wenn >> jemand ein WebUI machen will, sollte man das aber unbedingt in eine >> advanced Menü verpacken wo auch ne dicke Warnung dran steht, was schief >> laufen kann. > >> mfg >> >> Christian >> >>> >>> >>> Am 02.01.2018 um 13:20 schrieb Christian Dresel: >>>> To set Hoodconfiguration fix use: >>>> uci set fff.system.hoodfix=1 >>>> uci commit >>>> to revert this use: >>>> uci del fff.system.hoodfix >>>> uci commit >>>> >>>> Signed-off-by: Christian Dresel <fff@chrisi01.de> >>>> --- >>>> src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 5 +++++ >>>> 1 file changed, 5 insertions(+) >>>> >>>> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >>>> index 2b9c47e..02d9a90 100755 >>>> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >>>> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood >>>> @@ -1,5 +1,10 @@ >>>> #!/bin/sh >>>> >>>> +if [ $(uci -q get fff.system.hoodfix) ]; then >>>> + echo "Hood is fix, we do nothing" >>>> + exit 1 >>> Bei einem gewünschtem Ausstieg ohne Fehler wäre errorlevel 0 schöner. >>> >>> Robert >>>> +fi >>>> + >>>> . /usr/share/libubox/jshn.sh >>>> . /lib/functions/fff/keyxchange >>>> . /lib/functions/fff/network >
To set Hoodconfiguration fix use: uci set fff.system.hoodfix=1 uci commit to revert this use: uci del fff.system.hoodfix uci commit Signed-off-by: Christian Dresel <fff@chrisi01.de> --- src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 5 +++++ 1 file changed, 5 insertions(+)