[patch master] fix patching error by emty build_patches dir

Jan-Tarek Butt tarek at ring0.de
Sa Jul 23 23:46:08 CEST 2016


Ahoi Tim,

> Cool ein Patch von dir hier zu sehen.

Immer wieder gerne.

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

Ja das ist prinzipiell richtig, allerding wird beim meiner Variante nicht
auf die Existenz des dirs geprüft sondern auf Inhalt in einem dir.

> Die Einrückung im Script ist auf den ersten Blick 4 Spaces. Bitte
> innerhalb einer Datei drauf achten, dass es konsistent bleibt.

Pardon, kommt nach obiger klärung in version 2

vg
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/20160723/7f9cedec/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev