This causes package_$pkgname() to be preferred over package() in the non-split case, but the behavior if both functions exist was undocumented anyway. --- scripts/makepkg.sh.in | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index e9080a70..748481e4 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1481,30 +1481,28 @@ fi # check we have the software required to process the PKGBUILD check_software || exit $E_MISSING_MAKEPKG_DEPS

-if (( ${#pkgname[@]} > 1 )); then +if (( ${#pkgname[@]} > 1 )) || have_function package_${pkgname}; then SPLITPKG=1 fi

# test for available PKGBUILD functions if have_function prepare; then # "Hide" prepare() function if not going to be run if [[ $RUN_PREPARE != "n" ]]; then PREPAREFUNC=1 fi fi if have_function build; then BUILDFUNC=1 fi if have_function check; then # "Hide" check() function if not going to be run if [[ $RUN_CHECK = 'y' ]] || { ! check_buildenv "check" "n" && [[ $RUN_CHECK != "n" ]]; }; then CHECKFUNC=1 fi fi if have_function package; then PKGFUNC=1 -elif [[ $SPLITPKG -eq 0 ]] && have_function package_${pkgname}; then - SPLITPKG=1 fi

# check if gpg signature is to be created and if signing key is valid

Put this into a new function close to run_split_packaging so similar code is nearby. --- scripts/makepkg.sh.in | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 4684e444..f8f43540 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1121,6 +1121,18 @@ backup_package_variables() { done }

+run_solo_packaging() { + pkgdir="$pkgdirbase/$pkgname" + mkdir "$pkgdir" + if (( PKGFUNC )); then + run_package + fi + tidy_install + lint_package || exit $E_PACKAGE_FAILED + create_package + create_debug_package +} + run_split_packaging() { local pkgname_backup=("${pkgname[@]}") local restore_package_variables @@ -1537,15 +1549,7 @@ if (( INFAKEROOT )); then

chmod 755 "$pkgdirbase" if (( ! SPLITPKG )); then - pkgdir="$pkgdirbase/$pkgname" - mkdir "$pkgdir" - if (( PKGFUNC )); then - run_package - fi - tidy_install - lint_package || exit $E_PACKAGE_FAILED - create_package - create_debug_package + run_solo_packaging else run_split_packaging fi

Causes it to be reset (to $pkgdirbase/$pkgbase) between subpackages. This shouldn't be visible. --- scripts/makepkg.sh.in | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 84ae08db..27f627ac 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -774,13 +774,13 @@ create_package() { }

create_debug_package() { + local pkgdir="$pkgdirbase/$pkgbase-@DEBUGSUFFIX@" + # check if a debug package was requested if ! check_option "debug" "y" || ! check_option "strip" "y"; then return fi

- pkgdir="$pkgdirbase/$pkgbase-@DEBUGSUFFIX@" - # check if we have any debug symbols to package if dir_is_empty "$pkgdir/usr/lib/debug"; then return @@ -1122,7 +1122,7 @@ backup_package_variables() { }

run_solo_packaging() { - pkgdir="$pkgdirbase/$pkgname" + local pkgdir="$pkgdirbase/$pkgname" mkdir "$pkgdir" if [[ -n $1 ]] || (( PKGFUNC )); then run_package $1

$restoretrap is empty if the trap was not set. This caused the trap handler to remain and override later exit codes. --- scripts/makepkg.sh.in | 1 + 1 file changed, 1 insertion(+)

diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index ed0ceaec..3a3f4c30 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -432,6 +432,7 @@ run_function_safe() {

run_function "$1"

+ trap - ERR eval "$restoretrap" eval "$restoreset" eval "$restoreshopt"

On 01/06/18 02:24, Jan Alexander Steffens (heftig) wrote:

Pull out the expected=y/n check into a separate function and make use of the fact we can just concatenate the fallback arrays to get the same result.

---

Breaks using options=(!strip) in a PKGBUILD.

scripts/libmakepkg/util/option.sh.in | 85 +++++++++------------------- 1 file changed, 28 insertions(+), 57 deletions(-)

diff --git a/scripts/libmakepkg/util/option.sh.in b/scripts/libmakepkg/util/option.sh.in index 46e0568d..28357e27 100644 --- a/scripts/libmakepkg/util/option.sh.in +++ b/scripts/libmakepkg/util/option.sh.in @@ -48,95 +48,66 @@ in_opt_array() { }

+## +# usage : check_opt_array( $option, $expected_val, $haystack ) +# return : 0 - matches expected +# 1 - does not match expected +# 127 - not found +## +check_opt_array() {

• local option=$1 expected=$2; shift 2
• in_opt_array "$option" "$@"
• case $? in

• 0) # assert enabled
  

• 	[[ $expected = y ]]
  

• 	return ;;
  

• 1) # assert disabled
  

• 	[[ $expected = n ]]
  

• 	return ;;
  

• esac
• # not found
• return 127

+}

## # Checks to see if options are present in makepkg.conf or PKGBUILD; # PKGBUILD options always take precedence. # # usage : check_option( $option, $expected_val ) # return : 0 - matches expected # 1 - does not match expected # 127 - not found ## check_option() {

• in_opt_array "$1" ${options[@]}
• case $? in

• 0) # assert enabled
  

• 	[[ $2 = y ]]
  

• 	return ;;
  

• 1) # assert disabled
  

• 	[[ $2 = n ]]
  

• 	return
  

• esac
• # fall back to makepkg.conf options
• in_opt_array "$1" ${OPTIONS[@]}
• case $? in

• 0) # assert enabled
  

• 	[[ $2 = y ]]
  

• 	return ;;
  

• 1) # assert disabled
  

• 	[[ $2 = n ]]
  

• 	return
  

• esac
• # not found
• return 127

• check_opt_array "$@" "${options[@]}" "${OPTIONS[@]}"

These need switched.

}

## # Check if option is present in BUILDENV # # usage : check_buildenv( $option, $expected_val ) # return : 0 - matches expected # 1 - does not match expected # 127 - not found ## check_buildenv() {

• in_opt_array "$1" ${BUILDENV[@]}
• case $? in

• 0) # assert enabled
  

• 	[[ $2 = "y" ]]
  

• 	return ;;
  

• 1) # assert disabled
  

• 	[[ $2 = "n" ]]
  

• 	return ;;
  

• esac
• # not found
• return 127

• check_opt_array "$@" "${BUILDENV[@]}"

}

## # Checks to see if options are present in BUILDENV or PKGBUILD; # PKGBUILD options always take precedence. # # usage : check_buildoption( $option, $expected_val ) # return : 0 - matches expected # 1 - does not match expected # 127 - not found ## check_buildoption() {

• in_opt_array "$1" ${options[@]}
• case $? in

• 0) # assert enabled
  

• 	[[ $2 = y ]]
  

• 	return ;;
  

• 1) # assert disabled
  

• 	[[ $2 = n ]]
  

• 	return
  

• esac
• in_opt_array "$1" ${BUILDENV[@]}
• case $? in

• 0) # assert enabled
  

• 	[[ $2 = y ]]
  

• 	return ;;
  

• 1) # assert disabled
  

• 	[[ $2 = n ]]
  

• 	return
  

• esac
• # not found
• return 127

• check_opt_array "$@" "${options[@]}" "${BUILDENV[@]}"

These need switched.

}