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

[Eli Schwartz]

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 <eschwartz at archlinux.org>
---

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
thing.

 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"
 		fi
 	done
-- 
2.16.2