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

Adrian Schmutzler mail at adrianschmutzler.de
Fr Jul 27 10:07:49 CEST 2018


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




Mehr Informationen über die Mailingliste franken-dev