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

[Fabian Bläse]

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 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

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: Message signed with OpenPGP
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20171104/14b760c2/attachment.sig>