[PATCH 1/2] configurehood: Copy local hoodfile to common location

[Adrian Schmutzler]

Hallo Fabian,

auch wenn ich zunächst anderer Meinung war: Es ist so stimmiger. Das lokale Hoodfile wird einfach eine "Quelle" wie auch Gateway oder configap. Und alle 5 Minuten im (RAM-)Speicher schreiben tun wir bei den anderen Varianten ja auch.

Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>

Ein paar Anmerkungen/Fragen:
- Macht cp auto-overwrite?
- Ich überlege, was passiert, wenn so ein File gerade geschrieben (per wget) oder kopiert wird. Liegt dann da für 2 sec. ein halbes File rum oder macht das das Dateisystem intelligenter? (Man könnte ja auch hoodfile.tmp schreiben und dann mv machen) Für configurehood wäre es wurscht, aber ggf. würde dann ein anderes Skript dieses halbe File auswerten und versterben. (Mir fällt allerdings nichts "wichtiges" außerhalb configurehood ein, wo es schlimm wäre, wenn das ganz selten kaputt geht).

Da wir hier an configurehood rumfummeln, würde ich den Patch etwas länger liegen lassen vor dem applien (ne Woche statt 2 Tage), falls noch jemand was zu sagen hat.

Beste Grüße

Adrian



> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of
> Fabian Bläse
> Sent: Montag, 3. Dezember 2018 20:55
> To: franken-dev at freifunk.net
> Subject: [PATCH 1/2] configurehood: Copy local hoodfile to common location
> 
> Instead of overwriting local variable to local hoodfile location,
> copy it to the commonly used location. This makes the local hoodfile
> behave exactly like central hoodfiles, which prevents some border cases.
> 
> The previous behaviour mainly caused issues when using getJsonPath() and
> getUpgradePath() from fff-hoodutils.
> 
> Signed-off-by: Fabian Bläse <fabian at blaese.de>
> ---
>  src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> index de60c21..4ae1364 100755
> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> @@ -77,7 +77,7 @@ if [ -s "$hoodfilewww" ] && isGatewayAvailable ; then
>  fi
> 
>  if [ -s "$hoodfilelocal" ]; then
> -	hoodfiletmp="$hoodfilelocal"
> +	cp "$hoodfilelocal" "$hoodfiletmp"
>  	echo "Use local hood file"
>  else
>  	# if we have Internet, we download the Hoodfile from the
> keyxchangev2
> --
> 2.19.1
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20181206/76b04ef0/attachment.sig>