[patch master] fix patching error by emty build_patches dir
Jan-Tarek Butt
tarek at ring0.de
So Jul 24 00:54:27 CEST 2016
On 07/24/16 00:07, Robert wrote:
> Hi,
>
>
> Am 23.07.2016 um 21:36 schrieb Tim Niemeyer:
>> Hi Tarek
>>
>> Cool ein Patch von dir hier zu sehen.
>>
>> Am Samstag, den 23.07.2016, 21:18 +0200 schrieb Jan-Tarek Butt:
>>> Signed-off-by: Jan-Tarek Butt <tarek at ring0.de>
>>> ---
>>> buildscript | 2 ++
>>> 1 file changed, 2 insertions(+)
>>>
>>> diff --git a/buildscript b/buildscript
>>> index ae75027..53985c4 100755
>>> --- a/buildscript
>>> +++ b/buildscript
>>> @@ -122,9 +122,11 @@ get_source() {
>>> }
>>>
>>> patch_target() {
>>> + if test "$(ls -A "$PWD"||"/build_patches/openwrt/")"; then
>> Du startest drei Programme (oder bash build-in's). Mir würde "if test -d
>> "$DIR"; then" besser gefallen, als ein Aufruf von test welcher den
>> Ausdruck eigentlich nur durch reicht. Alternativ: "if [ -d "$DIR" ];
>> then".
> |Ein i||f["$(ls /path/to/dir)"];then ist schon ok. Es soll ja auf "leer"
> und nicht auf "exist" geprüft werden.
>
> Problem ist doch, dass der * bei leerem Verzeichnis (oder hier kein
> xxxxx.patch) nicht expandiert und ein * bleibt. Könnte man mit einem
> shopt -s nullglob ändern, dann braucht man das if nicht. (kann die ash
> auf unseren Routern aber nicht)
Im grunde bezieht sich das nicht auf die ash des Routers sondern in
erster Linie ist das eine build host Funktion.
>
> Darum würde ich gleich auf *.patch prüfen |
Jop, das ist korrekt. bzgl. der *.patch endung wäre deine Variante die bessere, um
letztlich direkt auf Patches zu prüfen.
>
> if test "$(ls -A "$PWD"||"/build_patches/openwrt/*.patch")"; then
>
> |Sonst läuft er wieder in die patch Zeile rein, wenn irgend eine Datei
> existiert.
cheers
Tarek
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname : signature.asc
Dateityp : application/pgp-signature
Dateigröße : 819 bytes
Beschreibung: OpenPGP digital signature
URL : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20160724/eef56d6c/attachment.sig>
Mehr Informationen über die Mailingliste franken-dev