[PATCH] Remove unusual shebang in files to be sourced.

[Adrian Schmutzler]

Hallo,

applied.

Grüße

Adrian

> -----Original Message-----
> From: Adrian Schmutzler [mailto:mail at adrianschmutzler.de]
> Sent: Montag, 10. September 2018 17:25
> To: 'robert' <rlanghammer at web.de>; 'franken-dev at freifunk.net' <franken-
> dev at freifunk.net>
> Subject: RE: [PATCH] Remove unusual shebang in files to be sourced.
> 
> Hallo Robert,
> 
> Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> 
> Grüße
> 
> Adrian
> 
> > -----Original Message-----
> > From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of
> > robert
> > Sent: Montag, 10. September 2018 01:19
> > To: franken-dev at freifunk.net
> > Subject: Re: [PATCH] Remove unusual shebang in files to be sourced.
> >
> > Hallo Adrian,
> >
> > du hast recht, der shebang wird nicht ausgewertet. Darum hat der da auch
> > nichts zu suchen. Solche Dateien bekommen auch kein executable.
> >
> > Wenn jemand eine Bibliothek nicht nur fuer sich schreibt, hat er die
> > posix sh konform zu halten. Wenn nicht, gibt es was auf die Finger, denn
> > das ist wie Reissnaegel auf den Stuhl des Nachfolgers zu legen. Wenn wir
> > es also richtig machen, kann und darf in so einem Kommentar nur /bin/sh
> > stehen, was man dann auch wieder weg lassen kann.
> >
> >
> > Die Dokumentation der Funktionen faende ich auch gut. Sollte vielleicht
> > der Autor in einem extra Patch machen.
> >
> > Robert
> >
> >
> > Am 27.07.2018 um 11:05 schrieb Adrian Schmutzler:
> > > Hallo,
> > >
> > > den Kompromiss mit dem Kommentar finde ich plausibel, wäre in meinem
> > Sinne (und ordentlicher).
> > >
> > > Zum Thema Dokumentation:
> > > Ich habe gewisse Hemmungen, die Dokumentation in diesen Files zu sehr
> > aufzublasen, da das ja alles (knappen) Speicher belegt. Eine gewisse Grund-
> > Doku sollte vorhanden sein, aber darüber hinaus bin ich mir nicht sicher, ob
> man
> > das da direkt reinschreiben muss.
> > > Ich habe allerdings auch keinen Überblick, inwiefern dass nach der
> > Komprimierung des Dateisystems und unter Berücksichtigung der block-size
> > noch relevant ist bzw. wie sich das da auswirkt. Normal sind Text-Dateien ja
> > super-komprimierbar.
> > >
> > > Beste Grüße
> > >
> > > Adrian
> > >
> > >> -----Original Message-----
> > >> From: Tim Niemeyer [mailto:tim at tn-x.org]
> > >> Sent: Freitag, 27. Juli 2018 10:55
> > >> To: Adrian Schmutzler <mail at adrianschmutzler.de>; franken-
> > dev at freifunk.net;
> > >> 'Robert Langhammer' <rlanghammer at web.de>
> > >> Subject: Re: [PATCH] Remove unusual shebang in files to be sourced.
> > >>
> > >> Am Freitag, den 27.07.2018, 10:07 +0200 schrieb Adrian Schmutzler:
> > >>> Hallo,
> > >>>
> > >>> ein Gedanke hierzu:
> > >>>
> > >>> Die Shebangs machen ja keinen Schaden, sie sind lediglich
> > >>> nutzlos/überflüssig.
> > >>>
> > >>> Ein Vorteil besteht meines Erachtens allerdings darin, dass man
> > >>> dadurch auch
> > >>> bei gesourceten Dateien auf einen Blick erkennen kann, für welche
> > >>> Shell sie
> > >>> geschrieben sind, ohne erst den Code zu analysieren.
> > >>>
> > >>> In ein paar Jahren kann so ein nicht mit der Sache befasster Leser
> > >>> des Codes
> > >>> schnell erkennen, für welche Shell die Dateien gesourced werden
> > >>> können (oder
> > >>> zumindest sollen). Das ist in meinen Augen zumindest praktisch und
> > >>> würde für
> > >>> mich eher dafür sprechen, die shebands drin zu lassen.
> > >> Das ist ein gutes Argument. Aber es zeigt gleichfalls auch auf, dass
> > >> man denken könnte die Datei ist zum direkten Ausführen da.
> > >>
> > >> Um beide Argumente zu befriedigen könnte man das Shebang in einen
> > >> reinen Code-Kommentar umbauen. Bei der Gelegenheit wäre es sinnvoll
> > >> gleich mal mehr Dokumentation in diese Dateien einzubauen.
> > >>
> > >> Tim
> > >>
> > >>> Das ist aber nur ein Gedanke und wenn ihr die rausmacht, finde ich
> > >>> das jetzt
> > >>> auch nicht schlimm.
> > >>>
> > >>> Beste Grüße
> > >>>
> > >>> Adrian
> > >>>
> > >>>
> > >>>
> > >>>> -----Original Message-----
> > >>>> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On
> > >>>> Behalf Of
> > >>>> Robert Langhammer
> > >>>> Sent: Mittwoch, 25. Juli 2018 15:48
> > >>>> To: franken-dev at freifunk.net
> > >>>> Subject: [PATCH] Remove unusual shebang in files to be sourced.
> > >>>>
> > >>>> Fixes: #101
> > >>>>
> > >>>> Signed-off-by: Robert Langhammer <rlanghammer at web.de>
> > >>>> ---
> > >>>>  src/packages/fff/fff-
> > >>>> hoodutils/files/lib/functions/fff/evalhoodinfo | 1 -
> > >>>>  src/packages/fff/fff-
> > >>>> hoodutils/files/lib/functions/fff/keyxchange   | 1 -
> > >>>>  src/packages/fff/fff-
> > >>>> network/files/lib/functions/fff/evalbytes      | 1 -
> > >>>>  src/packages/fff/fff-
> > >>>> network/files/lib/functions/fff/network        | 1 -
> > >>>>  src/packages/fff/fff-
> > >>>> timeserver/files/lib/functions/fff/timeserver  | 1 -
> > >>>>  src/packages/fff/fff-
> > >>>> wireless/files/lib/functions/fff/wireless      | 1 -
> > >>>>  6 files changed, 6 deletions(-)
> > >>>>
> > >>>> diff --git
> > >>> a/src/packages/fff/fff-hoodutils/files/lib/functions/fff/evalhoodinfo
> > >>>> b/src/packages/fff/fff-
> > >>>> hoodutils/files/lib/functions/fff/evalhoodinfo
> > >>>> index c487ea7..5b3bdb5 100644
> > >>>> --- a/src/packages/fff/fff-
> > >>>> hoodutils/files/lib/functions/fff/evalhoodinfo
> > >>>> +++ b/src/packages/fff/fff-
> > >>>> hoodutils/files/lib/functions/fff/evalhoodinfo
> > >>>> @@ -1,4 +1,3 @@
> > >>>> -#!/bin/sh
> > >>>>  # Copyright 2017 Adrian Schmutzler
> > >>>>  # License GPLv3
> > >>>>
> > >>>> diff --git
> > >>> a/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange
> > >>>> b/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange
> > >>>> index 30963ae..e39abb2 100644
> > >>>> --- a/src/packages/fff/fff-
> > >>>> hoodutils/files/lib/functions/fff/keyxchange
> > >>>> +++ b/src/packages/fff/fff-
> > >>>> hoodutils/files/lib/functions/fff/keyxchange
> > >>>> @@ -1,4 +1,3 @@
> > >>>> -#!/bin/sh
> > >>>>  # Copyright 2017 Adrian Schmutzler
> > >>>>  # License GPLv3
> > >>>>
> > >>>> diff --git a/src/packages/fff/fff-
> > >>>> network/files/lib/functions/fff/evalbytes
> > >>>> b/src/packages/fff/fff-network/files/lib/functions/fff/evalbytes
> > >>>> index e550381..e6e5d8b 100644
> > >>>> --- a/src/packages/fff/fff-
> > >>>> network/files/lib/functions/fff/evalbytes
> > >>>> +++ b/src/packages/fff/fff-
> > >>>> network/files/lib/functions/fff/evalbytes
> > >>>> @@ -1,4 +1,3 @@
> > >>>> -#!/bin/sh
> > >>>>
> > >>>>  human_bytes() {
> > >>>>  	while read B ; do
> > >>>> diff --git a/src/packages/fff/fff-
> > >>>> network/files/lib/functions/fff/network
> > >>>> b/src/packages/fff/fff-network/files/lib/functions/fff/network
> > >>>> index af9b343..06a6a84 100644
> > >>>> --- a/src/packages/fff/fff-network/files/lib/functions/fff/network
> > >>>> +++ b/src/packages/fff/fff-network/files/lib/functions/fff/network
> > >>>> @@ -1,4 +1,3 @@
> > >>>> -#!/bin/sh
> > >>>>  # Copyright 2017 Adrian Schmutzler
> > >>>>  # License GPLv3
> > >>>>
> > >>>> diff --git
> > >>> a/src/packages/fff/fff-timeserver/files/lib/functions/fff/timeserver
> > >>>> b/src/packages/fff/fff-
> > >>>> timeserver/files/lib/functions/fff/timeserver
> > >>>> index d25b9be..122e75a 100644
> > >>>> --- a/src/packages/fff/fff-
> > >>>> timeserver/files/lib/functions/fff/timeserver
> > >>>> +++ b/src/packages/fff/fff-
> > >>>> timeserver/files/lib/functions/fff/timeserver
> > >>>> @@ -1,4 +1,3 @@
> > >>>> -#!/bin/sh
> > >>>>  # Copyright 2016 Tim Niemeyer
> > >>>>  # License GPLv3
> > >>>>
> > >>>> diff --git a/src/packages/fff/fff-
> > >>>> wireless/files/lib/functions/fff/wireless
> > >>>> b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless
> > >>>> index 5b63397..abbfc93 100644
> > >>>> --- a/src/packages/fff/fff-
> > >>>> wireless/files/lib/functions/fff/wireless
> > >>>> +++ b/src/packages/fff/fff-
> > >>>> wireless/files/lib/functions/fff/wireless
> > >>>> @@ -1,4 +1,3 @@
> > >>>> -#!/bin/sh
> > >>>>  # Copyright 2016 Tim Niemeyer
> > >>>>  # License GPLv3
> > >>>>
> > >>>> --
> > >>>> 2.11.0
> > >>>
> >