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

[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