| Message ID | 20180725134751.12531-1-rlanghammer@web.de |
|---|---|
| State | Accepted |
| Headers | show |
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
Hi Das schaut gut aus. Reviewed-by: Tim Niemeyer <tim@tn-x.org> Tim Am Mittwoch, den 25.07.2018, 15:47 +0200 schrieb Robert Langhammer: > Fixes: #101 > > Signed-off-by: Robert Langhammer <rlanghammer@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 >
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@freifunk.net] On Behalf Of > Robert Langhammer > Sent: Mittwoch, 25. Juli 2018 15:48 > To: franken-dev@freifunk.net > Subject: [PATCH] Remove unusual shebang in files to be sourced. > > Fixes: #101 > > Signed-off-by: Robert Langhammer <rlanghammer@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
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@freifunk.net] On > > Behalf Of > > Robert Langhammer > > Sent: Mittwoch, 25. Juli 2018 15:48 > > To: franken-dev@freifunk.net > > Subject: [PATCH] Remove unusual shebang in files to be sourced. > > > > Fixes: #101 > > > > Signed-off-by: Robert Langhammer <rlanghammer@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 > >
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@tn-x.org] > Sent: Freitag, 27. Juli 2018 10:55 > To: Adrian Schmutzler <mail@adrianschmutzler.de>; franken-dev@freifunk.net; > 'Robert Langhammer' <rlanghammer@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@freifunk.net] On > > > Behalf Of > > > Robert Langhammer > > > Sent: Mittwoch, 25. Juli 2018 15:48 > > > To: franken-dev@freifunk.net > > > Subject: [PATCH] Remove unusual shebang in files to be sourced. > > > > > > Fixes: #101 > > > > > > Signed-off-by: Robert Langhammer <rlanghammer@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 > > > >
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@tn-x.org] >> Sent: Freitag, 27. Juli 2018 10:55 >> To: Adrian Schmutzler <mail@adrianschmutzler.de>; franken-dev@freifunk.net; >> 'Robert Langhammer' <rlanghammer@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@freifunk.net] On >>>> Behalf Of >>>> Robert Langhammer >>>> Sent: Mittwoch, 25. Juli 2018 15:48 >>>> To: franken-dev@freifunk.net >>>> Subject: [PATCH] Remove unusual shebang in files to be sourced. >>>> >>>> Fixes: #101 >>>> >>>> Signed-off-by: Robert Langhammer <rlanghammer@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 >>>
Moin, bzgl. Dokumentation: Man kann bei Textdateien locker mit Faktor 5 rechnen. Selbst wenn du da 10 KB Doku reinschreibst (und das ist ne Menge) produzierst du gerade mal 2 KB Daten - eher weniger. Wir haben auch eine Blockgröße von 512 KB, da gehe ich von deutlich größeren Kompressionsraten aus. Kurz: Wenn da nicht gerade jemand sprichwörtliche Romane schreibt ist es schlicht irrelevant. Da gibts deutlich lohnendere Stellen Platz zu sparen (hab nur mal ganz kurz drübergeschaut): * quashfs-Optionen durchgehen - da gibts noch einige leckere Dinge (z.B. xz in Verbindung mit dem arm-Filter) * Global build settings (core dumps raus, sysrq raus, swap raus, unnötige exports / funktionen strippen etc.) * wpad-mini raus * Wenn wir hardcore gehen wollen: JS/css/html/sh minimizer (das haut dann u.a. auch die Kommentare raus) Gerade letzteres würde ich aber nur dann machen, wenn es gar nicht mehr anders geht - und auch nur noch beim tiny Target, nicht aber beim generic. Viele Grüße, Michael Fritscher Am 2018-07-27 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@tn-x.org] >> Sent: Freitag, 27. Juli 2018 10:55 >> To: Adrian Schmutzler <mail@adrianschmutzler.de>; >> franken-dev@freifunk.net; >> 'Robert Langhammer' <rlanghammer@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@freifunk.net] On >> > > Behalf Of >> > > Robert Langhammer >> > > Sent: Mittwoch, 25. Juli 2018 15:48 >> > > To: franken-dev@freifunk.net >> > > Subject: [PATCH] Remove unusual shebang in files to be sourced. >> > > >> > > Fixes: #101 >> > > >> > > Signed-off-by: Robert Langhammer <rlanghammer@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 >> > >> >
Hallo Robert, Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> Grüße Adrian > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of > robert > Sent: Montag, 10. September 2018 01:19 > To: franken-dev@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@tn-x.org] > >> Sent: Freitag, 27. Juli 2018 10:55 > >> To: Adrian Schmutzler <mail@adrianschmutzler.de>; franken- > dev@freifunk.net; > >> 'Robert Langhammer' <rlanghammer@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@freifunk.net] On > >>>> Behalf Of > >>>> Robert Langhammer > >>>> Sent: Mittwoch, 25. Juli 2018 15:48 > >>>> To: franken-dev@freifunk.net > >>>> Subject: [PATCH] Remove unusual shebang in files to be sourced. > >>>> > >>>> Fixes: #101 > >>>> > >>>> Signed-off-by: Robert Langhammer <rlanghammer@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 > >>> >
Hallo, applied. Grüße Adrian > -----Original Message----- > From: Adrian Schmutzler [mailto:mail@adrianschmutzler.de] > Sent: Montag, 10. September 2018 17:25 > To: 'robert' <rlanghammer@web.de>; 'franken-dev@freifunk.net' <franken- > dev@freifunk.net> > Subject: RE: [PATCH] Remove unusual shebang in files to be sourced. > > Hallo Robert, > > Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > Grüße > > Adrian > > > -----Original Message----- > > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of > > robert > > Sent: Montag, 10. September 2018 01:19 > > To: franken-dev@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@tn-x.org] > > >> Sent: Freitag, 27. Juli 2018 10:55 > > >> To: Adrian Schmutzler <mail@adrianschmutzler.de>; franken- > > dev@freifunk.net; > > >> 'Robert Langhammer' <rlanghammer@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@freifunk.net] On > > >>>> Behalf Of > > >>>> Robert Langhammer > > >>>> Sent: Mittwoch, 25. Juli 2018 15:48 > > >>>> To: franken-dev@freifunk.net > > >>>> Subject: [PATCH] Remove unusual shebang in files to be sourced. > > >>>> > > >>>> Fixes: #101 > > >>>> > > >>>> Signed-off-by: Robert Langhammer <rlanghammer@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 > > >>> > >
Fixes: #101 Signed-off-by: Robert Langhammer <rlanghammer@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(-)