fff-hoods: Allow correct ula_prefix syntax in hoodfiles #90

fbl · 2021-01-27T12:40:32+01:00

fbl commented

2021-01-27 12:40:32 +01:00

In earlier firmware versions the prefix had to be written
in an incorrect syntax (missing a trailing colon).
To make hoodfiles with this old incorrect syntax work with
newer firmware versions like this one, we have to fix the
incorrect syntax. Both the old, incorrect and the correct
syntax work with this fix, so in the far away future, the
correct syntax can be used in hoodfiles.

This is based on and thus requires: #77

In earlier firmware versions the prefix had to be written in an incorrect syntax (missing a trailing colon). To make hoodfiles with this old incorrect syntax work with newer firmware versions like this one, we have to fix the incorrect syntax. Both the old, incorrect and the correct syntax work with this fix, so in the far away future, the correct syntax can be used in hoodfiles. This is based on and thus requires: #77

fbl added this to the 20210211-beta milestone 2021-01-27 12:40:32 +01:00

fbl referenced this pull request

2021-01-27 12:41:38 +01:00

Falsche IPv6-Syntax in Hoodfile #27

fbl commented

2021-01-27 17:51:54 +01:00

Changes:

Bump PKG_RELEASE

Changes: * Bump PKG_RELEASE

fbl force-pushed syntaxfix2 from 24ce581ca5 to 738cab0796

2021-01-27 17:52:03 +01:00

Compare

adschm reviewed 2021-01-27 20:05:56 +01:00

src/packages/fff/fff-hoods/files/usr/sbin/configurehood Outdated

						
				@ -198,0 +198,4 @@

						# newer firmware versions like this one, we have to fix the

						# incorrect syntax here. Both the old, incorrect and

						# the correct syntax work with this fix.

						prefix="$(echo "$prefix" | sed -e 's,\([^:]\):\/,\1::/,')"

adschm commented

2021-01-27 20:05:56 +01:00

Der forward-slash muss nicht escaped werden (es funktioniert offenbar mit und ohne backslash zum escapen). Da nicht notwendig würde ich aber die Variante ohne Backslash vorziehen:

prefix=$(echo "$prefix" | sed -e 's,$[^:]$:/,\1::/,')

Die äußeren Anführungszeichen können ebenfalls entfallen, die Funktion gibt es bei a=$() für umsonst.

Mit dem entfernten Backslash
Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>

Der forward-slash muss nicht escaped werden (es funktioniert offenbar mit und ohne backslash zum escapen). Da nicht notwendig würde ich aber die Variante ohne Backslash vorziehen: `prefix=$(echo "$prefix" | sed -e 's,$[^:]$:/,\1::/,')` Die äußeren Anführungszeichen können ebenfalls entfallen, die Funktion gibt es bei a=$() für umsonst. Mit dem entfernten Backslash `Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>`