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

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