freifunk-franken/firmware
9
7
Fork 14
Code Issues 48 Pull Requests 7 Releases 17 Wiki Activity

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

Closed
fbl wants to merge 1 commits from fbl:syntaxfix2 into master
pull from: fbl:syntaxfix2
merge into: freifunk-franken:master
Conversation 3 Commits 1 Files Changed 2 +10 -3
2 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/packages/fff/fff-hoods/Makefile
View File

@ -1,7 +1,7 @@
include $(TOPDIR)/rules.mk
PKG_NAME:=fff-hoods
PKG_RELEASE:=18
PKG_RELEASE:=19
include $(INCLUDE_DIR)/package.mk

11
src/packages/fff/fff-hoods/files/usr/sbin/configurehood
View File

@ -191,8 +191,15 @@ if [ -s "$hoodfiletmp" ]; then
if [ -n "$prefix" ] ; then
# remove escape character
prefix=$(echo "$prefix" | sed -e 's,\\,,')
# create proper prefix
prefix=$(echo "$prefix" | sed -e 's,/,:/,')
# 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 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
Outdated
Review
Copy Link

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>`
mac=$(cat "/sys/class/net/br-client/address")
addr=$(owipcalc "$prefix" add "::$(ipMacSuffix "$mac")")
addr_eui=$(owipcalc "$prefix" add "::$(ipEUISuffix "$mac")")