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

Mo Sep 10 09:28:59 CEST 2018

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