[v2] fff-hoods: fix sed to substitute timestamp

Submitted by Adrian Schmutzler on Nov. 2, 2017, 12:01 p.m.

Details

Message ID 1509624077-2596-1-git-send-email-freifunk@adrianschmutzler.de
State Accepted
Headers show

Commit Message

Adrian Schmutzler Nov. 2, 2017, 12:01 p.m. 
From: Robert Langhammer <rlanghammer@web.de>

Rebased (v2). Apply after fff-hoods: Use w5sta for download!

Signed-off-by: Robert Langhammer <rlanghammer@web.de>
Reviewed-by: Tim Niemeyer <tim@tn-x.org>
Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
Reviewed-by: Fabian Bläse <fabian@blaese.de>
Tested-by: Fabian Bläse <fabian@blaese.de>
---
 src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Patch hide | download patch | download mbox 

diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
index deba22b..8437c33 100755
--- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
+++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
@@ -140,8 +140,8 @@  if [ -s "$hoodfile" ]; then
 		json_get_var mesh_type5 mesh_type5
 	fi
 
-	catnew="$(cat "$hoodfile" | sed 's/"timestamp":[0-9]*/"timestamp":0/')"
-	catold="$(cat "$hoodfilecopy" 2>/dev/null | sed 's/"timestamp":[0-9]*/"timestamp":0/')"
+	catnew="$(cat "$hoodfile" | sed 's/"timestamp":"[0-9]*"/"timestamp":0/')"
+	catold="$(cat "$hoodfilecopy" 2>/dev/null | sed 's/"timestamp":"[0-9]*"/"timestamp":0/')"
 	sumnew=$(echo "$catnew" | sha256sum | cut -f1 -d " ")
 	sumold=$(echo "$catold" | sha256sum | cut -f1 -d " ")

Comments

Christian Dresel Nov. 2, 2017, 12:03 p.m. 
Hängt mal bitte die Reihenfolge mit dran, wie man der Reihe nach applien
soll, damit sich Tim leichter tut. Wenn ich es nicht verbummelt habe
sind wir bei:

[PATCH v8] fff-hoods: Use channel provided by gateway
[PATCH v5] fff-hoods: Introduce sector files for custom settings
[PATCH v3] consistently use same webserver port for hoodfile on wifi and lan
[PATCH v2] fff-hoods: Use variables for hood file names
[PATCH] fff-hoods: Use w5sta for download
und danach dieses hier.

Bitte nochmal um Bestätigung.

mfg

Christian

On 02.11.2017 13:01, Adrian Schmutzler wrote:
> From: Robert Langhammer <rlanghammer@web.de>
> 
> Rebased (v2). Apply after fff-hoods: Use w5sta for download!
> 
> Signed-off-by: Robert Langhammer <rlanghammer@web.de>
> Reviewed-by: Tim Niemeyer <tim@tn-x.org>
> Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> Reviewed-by: Fabian Bläse <fabian@blaese.de>
> Tested-by: Fabian Bläse <fabian@blaese.de>
> ---
>  src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> index deba22b..8437c33 100755
> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> @@ -140,8 +140,8 @@ if [ -s "$hoodfile" ]; then
>  		json_get_var mesh_type5 mesh_type5
>  	fi
>  
> -	catnew="$(cat "$hoodfile" | sed 's/"timestamp":[0-9]*/"timestamp":0/')"
> -	catold="$(cat "$hoodfilecopy" 2>/dev/null | sed 's/"timestamp":[0-9]*/"timestamp":0/')"
> +	catnew="$(cat "$hoodfile" | sed 's/"timestamp":"[0-9]*"/"timestamp":0/')"
> +	catold="$(cat "$hoodfilecopy" 2>/dev/null | sed 's/"timestamp":"[0-9]*"/"timestamp":0/')"
>  	sumnew=$(echo "$catnew" | sha256sum | cut -f1 -d " ")
>  	sumold=$(echo "$catold" | sha256sum | cut -f1 -d " ")
>  
>
Adrian Schmutzler Nov. 2, 2017, 12:05 p.m. 
Ja, ist korrekt, habs in der Commit-Message versteckt.

Gibt dann noch den Mini-Patch mit dem commit system, aber den kann man einfach zuletzt machen...

Grüße

Adrian

> -----Original Message-----
> From: Christian Dresel [mailto:fff@chrisi01.de]
> Sent: Donnerstag, 2. November 2017 13:03
> To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> dev@freifunk.net
> Subject: Re: [PATCH v2] fff-hoods: fix sed to substitute timestamp
> 
> Hängt mal bitte die Reihenfolge mit dran, wie man der Reihe nach applien
> soll, damit sich Tim leichter tut. Wenn ich es nicht verbummelt habe sind wir
> bei:
> 
> [PATCH v8] fff-hoods: Use channel provided by gateway [PATCH v5] fff-
> hoods: Introduce sector files for custom settings [PATCH v3] consistently use
> same webserver port for hoodfile on wifi and lan [PATCH v2] fff-hoods: Use
> variables for hood file names [PATCH] fff-hoods: Use w5sta for download
> und danach dieses hier.
> 
> Bitte nochmal um Bestätigung.
> 
> mfg
> 
> Christian
> 
> On 02.11.2017 13:01, Adrian Schmutzler wrote:
> > From: Robert Langhammer <rlanghammer@web.de>
> >
> > Rebased (v2). Apply after fff-hoods: Use w5sta for download!
> >
> > Signed-off-by: Robert Langhammer <rlanghammer@web.de>
> > Reviewed-by: Tim Niemeyer <tim@tn-x.org>
> > Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > Reviewed-by: Fabian Bläse <fabian@blaese.de>
> > Tested-by: Fabian Bläse <fabian@blaese.de>
> > ---
> >  src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > index deba22b..8437c33 100755
> > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> > @@ -140,8 +140,8 @@ if [ -s "$hoodfile" ]; then
> >  		json_get_var mesh_type5 mesh_type5
> >  	fi
> >
> > -	catnew="$(cat "$hoodfile" | sed 's/"timestamp":[0-
> 9]*/"timestamp":0/')"
> > -	catold="$(cat "$hoodfilecopy" 2>/dev/null | sed 's/"timestamp":[0-
> 9]*/"timestamp":0/')"
> > +	catnew="$(cat "$hoodfile" | sed 's/"timestamp":"[0-
> 9]*"/"timestamp":0/')"
> > +	catold="$(cat "$hoodfilecopy" 2>/dev/null | sed 's/"timestamp":"[0-
> 9]*"/"timestamp":0/')"
> >  	sumnew=$(echo "$catnew" | sha256sum | cut -f1 -d " ")
> >  	sumold=$(echo "$catold" | sha256sum | cut -f1 -d " ")
> >
> >
Tim Niemeyer Nov. 2, 2017, 7:57 p.m. 
Hi

Und applied.

Tim

Am Donnerstag, den 02.11.2017, 13:01 +0100 schrieb Adrian Schmutzler:
> From: Robert Langhammer <rlanghammer@web.de>
> 
> Rebased (v2). Apply after fff-hoods: Use w5sta for download!
> 
> Signed-off-by: Robert Langhammer <rlanghammer@web.de>
> Reviewed-by: Tim Niemeyer <tim@tn-x.org>
> Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> Reviewed-by: Fabian Bläse <fabian@blaese.de>
> Tested-by: Fabian Bläse <fabian@blaese.de>
> ---
>  src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> index deba22b..8437c33 100755
> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> @@ -140,8 +140,8 @@ if [ -s "$hoodfile" ]; then
>  		json_get_var mesh_type5 mesh_type5
>  	fi
>  
> -	catnew="$(cat "$hoodfile" | sed 's/"timestamp":[0-9]*/"timestamp":0/')"
> -	catold="$(cat "$hoodfilecopy" 2>/dev/null | sed 's/"timestamp":[0-9]*/"timestamp":0/')"
> +	catnew="$(cat "$hoodfile" | sed 's/"timestamp":"[0-9]*"/"timestamp":0/')"
> +	catold="$(cat "$hoodfilecopy" 2>/dev/null | sed 's/"timestamp":"[0-9]*"/"timestamp":0/')"
>  	sumnew=$(echo "$catnew" | sha256sum | cut -f1 -d " ")
>  	sumold=$(echo "$catold" | sha256sum | cut -f1 -d " ")
>  
> -- 
> 2.7.4
>
Fabian Blaese Nov. 4, 2017, 10:49 p.m. 
Mein Kommentar von der v1 ist irgendwie ein wenig abgesoffen..

> Der angegebene Regex geht kaputt, wenn man das JSON durch einen Beautifier jagt, da dann je nach Beautifier zwischen : und “[0-9]*” ein Leerzeichen auftauchen kann.
> Vielleicht sollte man das noch ergänzen: sed 's/"timestamp”: *"[0-9]*"/"timestamp":0/‘

Finde ich aktuell doch ein wenig unschön. Meinungen dazu?

Fabian

> On 2. Nov 2017, at 20:57, Tim Niemeyer <tim@tn-x.org> wrote:
> 
> Hi
> 
> Und applied.
> 
> Tim
> 
> Am Donnerstag, den 02.11.2017, 13:01 +0100 schrieb Adrian Schmutzler:
>> From: Robert Langhammer <rlanghammer@web.de>
>> 
>> Rebased (v2). Apply after fff-hoods: Use w5sta for download!
>> 
>> Signed-off-by: Robert Langhammer <rlanghammer@web.de>
>> Reviewed-by: Tim Niemeyer <tim@tn-x.org>
>> Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
>> Reviewed-by: Fabian Bläse <fabian@blaese.de>
>> Tested-by: Fabian Bläse <fabian@blaese.de>
>> ---
>> src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>> 
>> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
>> index deba22b..8437c33 100755
>> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
>> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
>> @@ -140,8 +140,8 @@ if [ -s "$hoodfile" ]; then
>> 		json_get_var mesh_type5 mesh_type5
>> 	fi
>> 
>> -	catnew="$(cat "$hoodfile" | sed 's/"timestamp":[0-9]*/"timestamp":0/')"
>> -	catold="$(cat "$hoodfilecopy" 2>/dev/null | sed 's/"timestamp":[0-9]*/"timestamp":0/')"
>> +	catnew="$(cat "$hoodfile" | sed 's/"timestamp":"[0-9]*"/"timestamp":0/')"
>> +	catold="$(cat "$hoodfilecopy" 2>/dev/null | sed 's/"timestamp":"[0-9]*"/"timestamp":0/')"
>> 	sumnew=$(echo "$catnew" | sha256sum | cut -f1 -d " ")
>> 	sumold=$(echo "$catold" | sha256sum | cut -f1 -d " ")
>> 
>> --
>> 2.7.4
>> 
> 
> --
> franken-dev mailing list
> franken-dev@freifunk.net
> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Tim Niemeyer Nov. 5, 2017, 9:12 a.m. 
Hi

Am 4. November 2017 23:49:11 MEZ schrieb "Fabian Bläse" <fabian@blaese.de>:
>Mein Kommentar von der v1 ist irgendwie ein wenig abgesoffen..

Japp. Sorry.

>> Der angegebene Regex geht kaputt, wenn man das JSON durch einen
>Beautifier jagt, da dann je nach Beautifier zwischen : und “[0-9]*” ein
>Leerzeichen auftauchen kann.
>> Vielleicht sollte man das noch ergänzen: sed 's/"timestamp”:
>*"[0-9]*"/"timestamp":0/‘
>
>Finde ich aktuell doch ein wenig unschön. Meinungen dazu?

Stimmt schon. Falls der Server irgendwann das json mal ein bisschen anders macht, wäre es doof.

Tim


>Fabian
>
>> On 2. Nov 2017, at 20:57, Tim Niemeyer <tim@tn-x.org> wrote:
>> 
>> Hi
>> 
>> Und applied.
>> 
>> Tim
>> 
>> Am Donnerstag, den 02.11.2017, 13:01 +0100 schrieb Adrian Schmutzler:
>>> From: Robert Langhammer <rlanghammer@web.de>
>>> 
>>> Rebased (v2). Apply after fff-hoods: Use w5sta for download!
>>> 
>>> Signed-off-by: Robert Langhammer <rlanghammer@web.de>
>>> Reviewed-by: Tim Niemeyer <tim@tn-x.org>
>>> Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
>>> Reviewed-by: Fabian Bläse <fabian@blaese.de>
>>> Tested-by: Fabian Bläse <fabian@blaese.de>
>>> ---
>>> src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>> 
>>> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
>b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
>>> index deba22b..8437c33 100755
>>> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
>>> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
>>> @@ -140,8 +140,8 @@ if [ -s "$hoodfile" ]; then
>>> 		json_get_var mesh_type5 mesh_type5
>>> 	fi
>>> 
>>> -	catnew="$(cat "$hoodfile" | sed
>'s/"timestamp":[0-9]*/"timestamp":0/')"
>>> -	catold="$(cat "$hoodfilecopy" 2>/dev/null | sed
>'s/"timestamp":[0-9]*/"timestamp":0/')"
>>> +	catnew="$(cat "$hoodfile" | sed
>'s/"timestamp":"[0-9]*"/"timestamp":0/')"
>>> +	catold="$(cat "$hoodfilecopy" 2>/dev/null | sed
>'s/"timestamp":"[0-9]*"/"timestamp":0/')"
>>> 	sumnew=$(echo "$catnew" | sha256sum | cut -f1 -d " ")
>>> 	sumold=$(echo "$catold" | sha256sum | cut -f1 -d " ")
>>> 
>>> --
>>> 2.7.4
>>> 
>> 
>> --
>> franken-dev mailing list
>> franken-dev@freifunk.net
>> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net