fff-hoods: Removed redundant json include

Submitted by Adrian Schmutzler on Nov. 1, 2017, 10:46 p.m.

Details

Message ID 1509576362-3567-1-git-send-email-freifunk@adrianschmutzler.de
State Rejected
Headers show

Commit Message

Adrian Schmutzler Nov. 1, 2017, 10:46 p.m. 
Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
---
 src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 1 -
 1 file changed, 1 deletion(-)

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 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
 . /lib/functions/fff/keyxchange
 . /lib/functions/fff/wireless
 . /lib/functions/fff/timeserver

Comments

Robert Langhammer Nov. 2, 2017, 8:05 a.m. 
Hallo,

ja, das sollte weg.

Reviewed-by:  Robert Langhammer rlanghammer@web.de


Am 01.11.2017 um 23:46 schrieb Adrian Schmutzler:
> Signed-off-by: Adrian Schmutzler <freifunk@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
>  . /lib/functions/fff/keyxchange
>  . /lib/functions/fff/wireless
>  . /lib/functions/fff/timeserver
Tim Niemeyer Nov. 2, 2017, 8:42 p.m. 
Hi

Am Mittwoch, den 01.11.2017, 23:46 +0100 schrieb Adrian Schmutzler:
> Signed-off-by: Adrian Schmutzler <freifunk@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
>
Adrian Schmutzler Nov. 2, 2017, 8:53 p.m. 
Hallo,

wenn so ein Doppel-Include mit Function-Redefinition kein Problem macht, können wir es auch drin lassen...
Das kann ich nicht umfassend beurteilen.

Grüße

Adrian

> -----Original Message-----
> From: Tim Niemeyer [mailto:tim@tn-x.org]
> Sent: Donnerstag, 2. November 2017 21:42
> To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> dev@freifunk.net
> Subject: Re: [PATCH] fff-hoods: Removed redundant json include
> 
> Hi
> 
> Am Mittwoch, den 01.11.2017, 23:46 +0100 schrieb Adrian Schmutzler:
> > Signed-off-by: Adrian Schmutzler <freifunk@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
> >
Tim Niemeyer Nov. 2, 2017, 8:59 p.m. 
Hi


$ cat test.sh
--- %< ---
#!/bin/busybox sh

. func.sh
. func.sh

myTest
--- >% ---

$ cat func.sh 
--- %< ---
myTest()
{
    echo "Yes, this is a test"
}
--- >% ---

$ chmod +x test.sh
$ ./test.sh
--- %< ---
Yes, this is a test
--- >% ---

Tim


Am Donnerstag, den 02.11.2017, 21:53 +0100 schrieb
mail@adrianschmutzler.de:
> Hallo,
> 
> wenn so ein Doppel-Include mit Function-Redefinition kein Problem macht, können wir es auch drin lassen...
> Das kann ich nicht umfassend beurteilen.
> 
> Grüße
> 
> Adrian
> 
> > -----Original Message-----
> > From: Tim Niemeyer [mailto:tim@tn-x.org]
> > Sent: Donnerstag, 2. November 2017 21:42
> > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> > dev@freifunk.net
> > Subject: Re: [PATCH] fff-hoods: Removed redundant json include
> > 
> > Hi
> > 
> > Am Mittwoch, den 01.11.2017, 23:46 +0100 schrieb Adrian Schmutzler:
> > > Signed-off-by: Adrian Schmutzler <freifunk@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
> > >
> 
>