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

Mo Sep 10 17:24:54 CEST 2018

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