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

[Tim Niemeyer]

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
> 
> 
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 488 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20180727/2e655a76/attachment.sig>