[patch master] fix patching error by emty build_patches dir
Robert
rlanghammer at web.de
So Jul 24 00:07:07 CEST 2016
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)
Darum würde ich gleich auf *.patch 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.
Robert
|
> Die Einrückung im Script ist auf den ersten Blick 4 Spaces. Bitte
> innerhalb einer Datei drauf achten, dass es konsistent bleibt.
>
> Tim
>
>> for patch in "$PWD"/build_patches/openwrt/*.patch; do
>> patch --no-backup-if-mismatch -p0 -d "$target" -i "$patch"
>> done
>> + fi
>> }
>>
>> prepare() {
>> --
>> 2.9.0
>>
>
>
-------------- 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/1df30391/attachment.sig>
Mehr Informationen über die Mailingliste franken-dev