Trying to strip multiple files installed to the same filepath (in different components of a split package) can produce surprising results if those files are built using different options. Per FS#63356 this is not a supported PKGBUILD configuration. Instead, warn about such cases when producing debug packages.
Fixes FS#63366 Signed-off-by: Eli Schwartz <[email protected]> --- v3: mention symbols are only included for the first copy scripts/libmakepkg/tidy/strip.sh.in | 20 +++++++++++++++++--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/scripts/libmakepkg/tidy/strip.sh.in b/scripts/libmakepkg/tidy/strip.sh.in index 301d1989..d69fd755 100644 --- a/scripts/libmakepkg/tidy/strip.sh.in +++ b/scripts/libmakepkg/tidy/strip.sh.in @@ -40,13 +40,21 @@ source_files() { awk '/DW_AT_name +:/{name=$8}/DW_AT_comp_dir +:/{{if (name == "<artificial>") next}{if (name !~ /^[<\/]/) {printf "%s/", $8}}{print name}}' } +# strip a file and append to global array "duplicate_buildids" if a file with +# an identical id was already stripped strip_file() { local binary=$1; shift if check_option "debug" "y"; then local bid=$(build_id "$binary") - # has this file already been stripped + # has this filepath been stripped before under a different Build Id + if [[ -f "$dbgdir/$binary.debug" ]]; then + if [[ -n $bid && $bid != $(build_id "$dbgdir/$binary.debug") ]]; then + duplicate_buildids+=("${binary#./}") + fi + fi + # has this file's hardlink already been stripped if [[ -n "$bid" ]]; then if [[ -f "$dbgdir/.build-id/${bid:0:2}/${bid:2}.debug" ]]; then return @@ -95,6 +103,7 @@ strip_file() { tidy_strip() { + local duplicate_buildids=() if check_option "strip" "y"; then msg2 "$(gettext "Stripping unneeded symbols from binaries and libraries...")" # make sure library stripping variables are defined to prevent excess stripping @@ -110,7 +119,7 @@ tidy_strip() { fi local binary strip_flags - find . -type f -perm -u+w -print0 2>/dev/null | while IFS= read -rd '' binary ; do + while IFS= read -rd '' binary ; do case "$(LC_ALL=C readelf -h "$binary" 2>/dev/null)" in *Type:*'DYN (Shared object file)'*) # Libraries (.so) or Relocatable binaries strip_flags="$STRIP_SHARED";; @@ -129,6 +138,11 @@ tidy_strip() { continue ;; esac strip_file "$binary" ${strip_flags} - done + done < <(find . -type f -perm -u+w -print0 2>/dev/null) + if (( ${#duplicate_buildids[@]} > 0 )); then + warning "Split packages contain multiple copies of the same named file but different Build Ids" + warning "Debug symbols are included for the first copy only" + printf '%s\n' "${duplicate_buildids[@]}" >&2 + fi fi } -- 2.25.0
