[[PATCH v5] 1/2] use custom names for binary builds. fixes #9

[mayosemmel]

Hallo Andreas,

grundsätzlich hast du bestimmt recht. Gehört meiner Meinung nach aber
nicht zu diesem Thema dazu. Der Patch ist so schon riesig genug.
Lass uns das lieber im Nachgang machen.

Grüße Jan

Am Freitag, den 11.03.2016, 11:29 +0100 schrieb A. Schulze:
> Jan Kraus:
> 
> > From: mojoaxel <alex at wunschik.net>
> >
> > Signed-off-by: mojoaxel <alex at wunschik.net>
> > Signed-off-by: Jan Kraus <mayosemmel at gmail.com>
> > ---
> >  board_prepare() {
> >      echo "nothing todo" > /dev/null
> >  }
> >
> >  board_prepare() {
> >      echo "nothing todo" > /dev/null
> >  }
> >
> >  board_prebuild() {
> >      echo "nothing todo" > /dev/null
> >  }
> 
> Hallo,
> 
> ich kann jetzt noch keinen diff bauen, aber beim Durchsehen euerer  
> Patches ist mir eine Sache aufgefallen,
> die ihr ev. Verbessern könnt / wollt.
> 
> Es geht um obige Funtionen. "echo 'foo' > /dev/null" sieht ja schon  
> irgendwie sinnfrei aus.
> Es muss aber valider Shellcode im Funktionsbody stehen.
> 
> Ich würde solche Funktionen daher so schreiben:
> 
>      # Kommentar, warum dieser Funktionsbody absichtlich leer ist
>      foo() {
>        :
>      }
> 
> Der ":" gilt als Shell-Kommando "no-op" und sollte von jeder  
> POSIX-konformen Shell implementiert werden.
> ( http://tldp.org/LDP/abs/html/special-chars.html#COLON0REF )
> 
> Mit der Busybox-Shell auf meinem Router habe ich's gerade getestet.
> 
> 
> Andreas
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 473 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20160311/605421d4/attachment-0002.sig>