| Message ID | 1500557370-2185-1-git-send-email-freifunk@adrianschmutzler.de |
|---|---|
| State | Superseded |
| Headers | show |
diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html index bdbd69b..7488af6 100755 --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html @@ -14,7 +14,7 @@ if [ "$REQUEST_METHOD" == "POST" ] ; then else # Allgemeine Einstellungen uci -q set "system.@system[0].hostname=${POST_hostname}" - uci -q get 'system.@system[0].hostname' > /proc/sys/kernel/hostname + echo $(uci -q get "system.@system[0].hostname") > /proc/sys/kernel/hostname uci -q set "system.@system[0].description=${POST_description}" uci -q set "system.@system[0].latitude=${POST_latitude}" uci -q set "system.@system[0].longitude=${POST_longitude}"
What does this change? Am 20.07.2017 um 15:29 schrieb Adrian Schmutzler: > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > --- > src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html > index bdbd69b..7488af6 100755 > --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html > +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html > @@ -14,7 +14,7 @@ if [ "$REQUEST_METHOD" == "POST" ] ; then > else > # Allgemeine Einstellungen > uci -q set "system.@system[0].hostname=${POST_hostname}" > - uci -q get 'system.@system[0].hostname' > /proc/sys/kernel/hostname > + echo $(uci -q get "system.@system[0].hostname") > /proc/sys/kernel/hostname > uci -q set "system.@system[0].description=${POST_description}" > uci -q set "system.@system[0].latitude=${POST_latitude}" > uci -q set "system.@system[0].longitude=${POST_longitude}" >
Ich weiß nicht, ob es bei bin/sh anders ist, aber log dich mal auf einem Router ein und probiere beides in der Konsole aus. Nur die zweite Variante funktioniert bei mir. -----Original Message----- From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of Dominik Heidler Sent: Dienstag, 25. Juli 2017 00:12 To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-dev@freifunk.net Subject: Re: [PATCH v2 2/2] fff-web: Fix setting hostname in settings.html What does this change? Am 20.07.2017 um 15:29 schrieb Adrian Schmutzler: > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > --- > src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html > index bdbd69b..7488af6 100755 > --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html > +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html > @@ -14,7 +14,7 @@ if [ "$REQUEST_METHOD" == "POST" ] ; then > else > # Allgemeine Einstellungen > uci -q set "system.@system[0].hostname=${POST_hostname}" > - uci -q get 'system.@system[0].hostname' > /proc/sys/kernel/hostname > + echo $(uci -q get "system.@system[0].hostname") > /proc/sys/kernel/hostname > uci -q set "system.@system[0].description=${POST_description}" > uci -q set "system.@system[0].latitude=${POST_latitude}" > uci -q set "system.@system[0].longitude=${POST_longitude}" >
Update: Hab mir das gerade noch mal genauer angesehen: Bei FW 20170110 machen beide Varianten keinen Unterschied, aber mit meiner recht neuen LEDE 17.01.2-Firmware funktioniert nur die Variante mit echo (auch in /bin/sh). Bei der Variante ohne echo kommt "root@?" raus. Hab jetzt keine ältere alpha zum Testen da, wie es sich dort verhält. Die Ausgabe wenn man nur "uci -q get ..." ohne Umleitung macht erfolgt bei beiden. Ist also evtl. ein Bug, der aber mit dem Patch ohne Nachteile behoben würde. -----Original Message----- From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of Dominik Heidler Sent: Dienstag, 25. Juli 2017 00:12 To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-dev@freifunk.net Subject: Re: [PATCH v2 2/2] fff-web: Fix setting hostname in settings.html What does this change? Am 20.07.2017 um 15:29 schrieb Adrian Schmutzler: > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > --- > src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html > index bdbd69b..7488af6 100755 > --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html > +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html > @@ -14,7 +14,7 @@ if [ "$REQUEST_METHOD" == "POST" ] ; then > else > # Allgemeine Einstellungen > uci -q set "system.@system[0].hostname=${POST_hostname}" > - uci -q get 'system.@system[0].hostname' > /proc/sys/kernel/hostname > + echo $(uci -q get "system.@system[0].hostname") > /proc/sys/kernel/hostname > uci -q set "system.@system[0].description=${POST_description}" > uci -q set "system.@system[0].latitude=${POST_latitude}" > uci -q set "system.@system[0].longitude=${POST_longitude}" >
Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> --- src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)