[PATCH] fff-hoods: Removed redundant json include

Tim Niemeyer tim at tn-x.org
Do Nov 2 21:42:04 CET 2017


Hi

Am Mittwoch, den 01.11.2017, 23:46 +0100 schrieb Adrian Schmutzler:
> Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> ---
>  src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 1 -
>  1 file changed, 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 981f7aa..e10d7d9 100755
> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> @@ -1,6 +1,5 @@
>  #!/bin/sh
>  
> -. /usr/share/libubox/jshn.sh
Ich möchte das eigentlich lieber da lassen, denn die Funktionen aus dem
File werden in configurehood nun mal gebraucht. Lässt man es weg, muss
man sich darauf verlassen, dass die functions/keyxchange das "ansaugt".
Sollte das functions/keyxchange aber mal eine andere Implementierung
nutzen, stimmt diese Annahme nicht mehr.

In dem Fall von jshn werden eh nur Funktionen bereit gestellt. Die
werden dann einmal überschrieben, sollte kein Problem sein, oder?

Wenn man mal Code in einem dot-Script ausführen möchte, müsste man das
dot-Script natürlich mit einem "Include"-Guard ausstatten. Das ist hier
aber nicht der Fall.

Tim

>  . /lib/functions/fff/keyxchange
>  . /lib/functions/fff/wireless
>  . /lib/functions/fff/timeserver
> -- 
> 2.7.4
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 473 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20171102/f020532e/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev