| Message ID | 1513944625-2149-1-git-send-email-freifunk@adrianschmutzler.de |
|---|---|
| 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 e6a47df..31ecffa 100755 --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood @@ -270,6 +270,8 @@ if [ -s "$hoodfile" ]; then cp "$hoodfile" "$hoodfilecopy" [ -s "$sectortmp" ] && cp "$sectortmp" "$sectorcopy" + # This is a workaround to enable alfred on devices which do not see a configap during initial setup + /etc/init.d/alfred restart else echo "We have no new file. We do nothing. We try it again in 5 minutes..."; fi
Hi Als Workaround wäre es erstmal ok, eigentlich müssten wir dazu aber noch die Abhängigkeit zu fff-alfred im Package modellieren. Unter der Voraussetzung, dass der Patch reverted wird, wenn br-mesh sauber angelegt wird: Reviewed-by: Tim Niemeyer <tim@tn-x.org> Tim Am Freitag, den 22.12.2017, 13:10 +0100 schrieb Adrian Schmutzler: > This is a workaround to enable alfred on routers which do not see > a configap during initial boot. > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > --- > > Although this might not be popular, it should fix the problem > perfectly. Since I do not think we solve the underlying issue > shortly, I think this patch should help in the meantime. > --- > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index e6a47df..31ecffa 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -270,6 +270,8 @@ if [ -s "$hoodfile" ]; then > cp "$hoodfile" "$hoodfilecopy" > [ -s "$sectortmp" ] && cp "$sectortmp" "$sectorcopy" > > + # This is a workaround to enable alfred on devices > which do not see a configap during initial setup > + /etc/init.d/alfred restart > else > echo "We have no new file. We do nothing. We try it > again in 5 minutes..."; > fi > -- > 2.7.4 >
Hallo Tim, "Unter der Voraussetzung, dass der Patch reverted wird, wenn br-mesh sauber angelegt wird:" Das geht nur so. Wenn du kein verbundenes Kabel hast und zum Boot kein configap verfügbar ist, dann gibt es kein "aktives" Interface im br-mesh. Und dann kommt das auch nicht hoch. Da müsste man dann entweder ein Dummy-WLAN-Interface anlegen oder die eth Interface irgendwie ohne Kabel aktiv machen. Grüße Adrian > -----Original Message----- > From: Tim Niemeyer [mailto:tim@tn-x.org] > Sent: Samstag, 20. Januar 2018 12:56 > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken- > dev@freifunk.net > Subject: Re: [PATCH] fff-hoods: Restart alfred after keyxchange file update > > Hi > > Als Workaround wäre es erstmal ok, eigentlich müssten wir dazu aber noch > die Abhängigkeit zu fff-alfred im Package modellieren. > > Unter der Voraussetzung, dass der Patch reverted wird, wenn br-mesh > sauber angelegt wird: > Reviewed-by: Tim Niemeyer <tim@tn-x.org> > > Tim > > > Am Freitag, den 22.12.2017, 13:10 +0100 schrieb Adrian Schmutzler: > > This is a workaround to enable alfred on routers which do not see a > > configap during initial boot. > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > > > --- > > > > Although this might not be popular, it should fix the problem > > perfectly. Since I do not think we solve the underlying issue shortly, > > I think this patch should help in the meantime. > > --- > > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > > index e6a47df..31ecffa 100755 > > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > > @@ -270,6 +270,8 @@ if [ -s "$hoodfile" ]; then > > cp "$hoodfile" "$hoodfilecopy" > > [ -s "$sectortmp" ] && cp "$sectortmp" "$sectorcopy" > > > > + # This is a workaround to enable alfred on devices > > which do not see a configap during initial setup > > + /etc/init.d/alfred restart > > else > > echo "We have no new file. We do nothing. We try it again in > 5 > > minutes..."; > > fi > > -- > > 2.7.4 > >
Am Samstag, den 20.01.2018, 13:01 +0100 schrieb mail@adrianschmutzler.de: > Hallo Tim, > > "Unter der Voraussetzung, dass der Patch reverted wird, wenn br-mesh > sauber angelegt wird:" > > Das geht nur so. Nein, und das ist so auch nicht sauber. Wie auch in https://mantis.freifunk-franken.de/view.php?id=78 beschrieben. > Wenn du kein verbundenes Kabel hast und zum Boot kein configap > verfügbar ist, dann gibt es kein "aktives" Interface im br-mesh. Und > dann kommt das auch nicht hoch. Ja, normal ist das so. Aber es gibt bestimmt einen Weg die Bridge immer da zu haben. > Da müsste man dann entweder ein Dummy-WLAN-Interface anlegen oder die > eth Interface irgendwie ohne Kabel aktiv machen. Ich würde einfach ein dummy Interface bevorzugen. Auf die Kabelzustände hat man dank der unterschiedlichen Switche keinen Einfluss. Wobei man da ggfs mit force_link arbeiten könnte, sofern OpenWRT das korrekt umgesetzt hat. Die WLAN Interfaces würde ich da komplett raus halten. Tim > Grüße > > Adrian > > > -----Original Message----- > > From: Tim Niemeyer [mailto:tim@tn-x.org] > > Sent: Samstag, 20. Januar 2018 12:56 > > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken- > > dev@freifunk.net > > Subject: Re: [PATCH] fff-hoods: Restart alfred after keyxchange > > file update > > > > Hi > > > > Als Workaround wäre es erstmal ok, eigentlich müssten wir dazu aber > > noch > > die Abhängigkeit zu fff-alfred im Package modellieren. > > > > Unter der Voraussetzung, dass der Patch reverted wird, wenn br-mesh > > sauber angelegt wird: > > Reviewed-by: Tim Niemeyer <tim@tn-x.org> > > > > Tim > > > > > > Am Freitag, den 22.12.2017, 13:10 +0100 schrieb Adrian Schmutzler: > > > This is a workaround to enable alfred on routers which do not see > > > a > > > configap during initial boot. > > > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > > > > > --- > > > > > > Although this might not be popular, it should fix the problem > > > perfectly. Since I do not think we solve the underlying issue > > > shortly, > > > I think this patch should help in the meantime. > > > --- > > > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/src/packages/fff/fff- > > > hoods/files/usr/sbin/configurehood > > > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > > > index e6a47df..31ecffa 100755 > > > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > > > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > > > @@ -270,6 +270,8 @@ if [ -s "$hoodfile" ]; then > > > cp "$hoodfile" "$hoodfilecopy" > > > [ -s "$sectortmp" ] && cp "$sectortmp" > > > "$sectorcopy" > > > > > > + # This is a workaround to enable alfred on > > > devices > > > which do not see a configap during initial setup > > > + /etc/init.d/alfred restart > > > else > > > echo "We have no new file. We do nothing. We try > > > it again in > > > > 5 > > > minutes..."; > > > fi > > > -- > > > 2.7.4 > > > > >
Hi Wie besprochen applied und ein neues Ticket eröffnet um den Workaround später nochmal vertiefen zu können: https://mantis.freifunk-franken.de/view.php?id=87 Tim Am Freitag, den 22.12.2017, 13:10 +0100 schrieb Adrian Schmutzler: > This is a workaround to enable alfred on routers which do not see > a configap during initial boot. > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > --- > > Although this might not be popular, it should fix the problem > perfectly. Since I do not think we solve the underlying issue > shortly, I think this patch should help in the meantime. > --- > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index e6a47df..31ecffa 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -270,6 +270,8 @@ if [ -s "$hoodfile" ]; then > cp "$hoodfile" "$hoodfilecopy" > [ -s "$sectortmp" ] && cp "$sectortmp" "$sectorcopy" > > + # This is a workaround to enable alfred on devices > which do not see a configap during initial setup > + /etc/init.d/alfred restart > else > echo "We have no new file. We do nothing. We try it > again in 5 minutes..."; > fi > -- > 2.7.4 >
This is a workaround to enable alfred on routers which do not see a configap during initial boot. Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> --- Although this might not be popular, it should fix the problem perfectly. Since I do not think we solve the underlying issue shortly, I think this patch should help in the meantime. --- src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 2 ++ 1 file changed, 2 insertions(+)