[PATCH v2] buildscript: copy all squashfs images for one device

[Fabian Bläse]

Find ich gut. Das hier ersetzt dann meinen Patch, der die tftp images hinzugefügt hat.
Beim applien bitte dran denken, dass das einen Konflikt mit den neuen Geräten (C50, 841c13) gibt.

Reviewed-by: Fabian Bläse <fabian at blaese.de>

Kleinkram inline.

Gruß
Fabian

On 02.04.20 16:23, Adrian Schmutzler wrote:
> diff --git a/buildscript b/buildscript
> index b0935a4c..3aa262db 100755
> --- a/buildscript
> +++ b/buildscript
> @@ -292,18 +292,22 @@ cp_firmware() {
>  
>      mkdir -p "bin/$variant"
>  
> +    imagesrcpath="$builddir/bin/targets/${chipset}/${subtarget}"
> +    imagedestpath="./bin/$variant"
> +
>      for image in ${images[@]}; do
> -        filename_build=${image//openwrt/fff-${version}}
> -        filename_build=${filename_build//squashfs-/}
> -        filename_build=${filename_build//${chipset}-${subtarget}-/}
> -        cp "$builddir/bin/targets/${chipset}/${subtarget}/$image" "./bin/$variant/$filename_build"
> -
> -        for region in "" "-eu" "-us"; do
> -            image_factory=${image/sysupgrade/factory$region}
> -            if [[ -f "$builddir/bin/targets/${chipset}/${subtarget}/$image_factory" ]]; then
> -              filename_build_factory=${filename_build/sysupgrade/factory$region}
> -              cp "$builddir/bin/targets/${chipset}/${subtarget}/$image_factory" "./bin/$variant/$filename_build_factory"
> -            fi
> +        # This assumes the set of images names is given with a wildcard
> +        fullsrcpath=$imagesrcpath/$image
fullsrcpattern= ?

> +        if ! ls $fullsrcpath >/dev/null 2>/dev/null; then
> +            echo "Warning: $image not found."
> +            continue
> +        fi
> +        for f in $fullsrcpath; do
> +            filename_build=${f##*/}
Kann man so machen, ich würde aber eher zu $(basename "$f") tendieren.

> +            filename_build=${filename_build//openwrt/fff-${version}}
> +            filename_build=${filename_build//squashfs-/}
> +            filename_build=${filename_build//${chipset}-${subtarget}-/}
> +            cp "$f" "$imagedestpath/$filename_build"
>          done
>      done
>  }
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: OpenPGP digital signature
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20200415/7ff563c8/attachment.sig>