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 <[email protected]>
> ---
> 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 |
