Re: [ptxdist] [PATCH] ptxd_make_xpkg_pkg.sh: Prevent incomplete comp variable and build break

[Michael Olbrich <m.olbrich@pengutronix.de>]

On Tue, Dec 10, 2024 at 09:15:51AM +0100, Carsten Schlote wrote:
> Changed code to detect broken detection of debug compression type. This
> prevents an invalid comp variable ('='), which in turn causes some
> abort, when objcopy is called with incomplete arguments.
> 
> If the compression can't be set and the comp variable is just "=", it
> will be set to an empty string. The target objcopy tool will use its
> internal default for compression in this case.
> 
> Signed-off-by: Carsten Schlote <schlote@vahanus.net>
> ---
>  scripts/lib/ptxd_make_xpkg_pkg.sh | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/scripts/lib/ptxd_make_xpkg_pkg.sh
> b/scripts/lib/ptxd_make_xpkg_pkg.sh
> index 08a0ab593..36bb3a564 100644
> --- a/scripts/lib/ptxd_make_xpkg_pkg.sh
> +++ b/scripts/lib/ptxd_make_xpkg_pkg.sh
> @@ -390,11 +390,15 @@ ptxd_install_compression_format() {
> 
>      libc="$(ptxd_cross_cc -print-file-name=libc.so.6 2> /dev/null)"
>      if [ -n "${libc}" ]; then
> -    comp="=$(readelf -t "${libc}" | sed -n -e
> '/COMPRESSED/{N;s/.*\(ZLIB\|ZSTD\).*/\1/p;q}' | tr '[:upper:]' '[:lower:]')"
> +        comp="=$(readelf -t "${libc}" | sed -n -e
> '/COMPRESSED/{N;s/.*\(ZLIB\|ZSTD\).*/\1/p;q}' | tr '[:upper:]' '[:lower:]')"

I think, if you do this (only add the '=' if s/// finds a match):

        comp="$(readelf -t "${libc}" | sed -n -e '/COMPRESSED/{N;s/.*\(ZLIB\|ZSTD\).*/=\1/p;q}' | tr '[:upper:]' '[:lower:]')"

> +    fi
> +    if [ "${comp}" == "=" ]; then
> +        comp=""

Then you don't need tis.

>      fi
>      export ptxd_install_file_objcopy_args="--only-keep-debug
> --compress-debug-sections${comp}"

Something (Thunderbird?) added extra new-lines that broke the patch.
Please send again. If you cannot use 'git send-email' then sending as
attachment should work as well.

>  }
> 
> +

Unrelated whitespace change.

Michael

>  ptxd_install_compression_format
> 
>  ptxd_install_file_extract_debug() {
> 
> 
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |