[PATCH 1/5] buildscript: support multiple patches on feeds

Steffen Pankratz kratz00 at gmx.de
Di Okt 13 18:20:26 CEST 2015


On Sun, 11 Oct 2015 21:35:27 +0200
Tim Niemeyer <tim.niemeyer at mastersword.de> wrote:

Hi Tim

Gute Idee.

Eine kleine Aenderung wuerde ich noch vorschlagen,
statt
count=$(( $count + 1 ))
wuerde ich
count=$((count + 1))
verwenden.
Das '$' braucht es nicht und koennte auch zu Seiteneffekten fuehren,
da der Ausdruck vorher expandiert werden wuerde.


Gruss
-Steffen

> Signed-off-by: Tim Niemeyer <tim.niemeyer at mastersword.de>
> ---
> 
>  buildscript | 20 ++++++++++++++------
>  1 file changed, 14 insertions(+), 6 deletions(-)
> 
> diff --git a/buildscript b/buildscript
> index 3611810..1f4450a 100755
> --- a/buildscript
> +++ b/buildscript
> @@ -7,7 +7,7 @@ OPENWRTURL="git://git.openwrt.org/14.07/openwrt.git"
>  PACKAGEREV="de5e37ac5be3d44c335faa9652bf4ce15784f4ad"
>  PACKAGEURL="https://github.com/openwrt/packages.git"
>  
> -## Feed definition [0]: name aka directory [1]: url [2]: revision [3]: patches
> +## Feed definition [0]: name aka directory, [1]: url, [2]: revision, [3..n]: patches
>  
>  #official openwrt packages
>  OPENWRT=(openwrt
> @@ -82,15 +82,23 @@ get_source() {
>          local NAME=${FEED[0]}
>          local URL=${FEED[1]}
>          local REV=${FEED[2]}
> -        local PATCH=${FEED[3]}
>  
>          checkout_git "$NAME" "$URL" "$REV"
> +
>          # Patches for feeds could be stored in known directories like build_patches/$NAME/
>          # That way multiple patches for one feed could be supported
> -        if [ ! -z "$PATCH" ] ; then
> -            echo "Patching $PATCH"
> -            git -C "$NAME" am --whitespace=nowarn "$PATCH"
> -        fi
> +        count=3
> +        while [ "x${FEED[count]}" != "x" ]
> +        do
> +            local PATCH=${FEED[count]}
> +            if [ ! -z "$PATCH" ] ; then
> +                echo "Patching $PATCH"
> +                git -C "$NAME" am --whitespace=nowarn "$PATCH"
> +            else
> +                echo "Warning, $PATCH not found."
> +            fi
> +            count=$(( $count + 1 ))
> +        done
>      done
>  
>      cd .. # packages
> -- 
> 2.1.4
> 
> -- 
> franken-dev mailing list
> franken-dev at freifunk.net
> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net


-- 
Hermes powered by Manjaro Linux (Linux 4.2.3)

Best regards, Steffen Pankratz.
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 181 bytes
Beschreibung: OpenPGP digital signature
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20151013/6c4028ed/attachment-0002.sig>


Mehr Informationen über die Mailingliste franken-dev