[PATCH] makepkg --packagelist: fix listing debug packages with makepkg.conf

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view

[PATCH] makepkg --packagelist: fix listing debug packages with makepkg.conf

Eli Schwartz-2
print_all_package_names used in_opt_array to check whether or not the
PKGBUILD itself has options=('debug' 'split') -- while checking to see
if it was enabled per split package which doesn't make sense as these
options apply globally. This prevented debug packages from being listed
if enabled via makepkg.conf rather than per PKGBUILD.

Instead, use check_option to determine whether makepkg actually thinks
it is meant to try creating a split debug package.

Signed-off-by: Eli Schwartz <[hidden email]>

This seems like a good idea, considering that --packagelist is meant to
be run in real world situations where makepkg.conf exists and is a

 scripts/libmakepkg/util/pkgbuild.sh.in | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/scripts/libmakepkg/util/pkgbuild.sh.in b/scripts/libmakepkg/util/pkgbuild.sh.in
index 6cfda0b3..b2684dde 100644
--- a/scripts/libmakepkg/util/pkgbuild.sh.in
+++ b/scripts/libmakepkg/util/pkgbuild.sh.in
@@ -150,9 +150,8 @@ print_all_package_names() {
  local architecture pkg opts a
  for pkg in ${pkgname[@]}; do
  architecture=$(get_pkg_arch $pkg)
- get_pkgbuild_attribute "$pkg" 'options' 1 opts
  printf "%s/%s-%s-%s%s\n" "$PKGDEST" "$pkg" "$version" "$architecture" "$PKGEXT"
- if in_opt_array "debug" ${opts[@]} && in_opt_array "strip" ${opts[@]}; then
+ if check_option "debug" "y" && check_option "strip" "y"; then
  printf "%s/%s-%s-%s-%s%s\n" "$PKGDEST" "$pkg" "@DEBUGSUFFIX@" "$version" "$architecture" "$PKGEXT"