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

[Tim Niemeyer]

Hi

Am 4. November 2017 23:49:11 MEZ schrieb "Fabian Bläse" <fabian at 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 at tn-x.org> wrote:
>> 
>> Hi
>> 
>> Und applied.
>> 
>> Tim
>> 
>> Am Donnerstag, den 02.11.2017, 13:01 +0100 schrieb Adrian Schmutzler:
>>> From: Robert Langhammer <rlanghammer at web.de>
>>> 
>>> Rebased (v2). Apply after fff-hoods: Use w5sta for download!
>>> 
>>> Signed-off-by: Robert Langhammer <rlanghammer at web.de>
>>> Reviewed-by: Tim Niemeyer <tim at tn-x.org>
>>> Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
>>> Reviewed-by: Fabian Bläse <fabian at blaese.de>
>>> Tested-by: Fabian Bläse <fabian at 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 at freifunk.net
>> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net