[pacman-dev] [PATCHv4 3/3] makepkg: Comment on install_package() return code
Pierre Neidhardt
ambrevar at gmail.com
Wed Mar 5 04:45:22 EST 2014
Previously, install_package() return code was either used as exit code or
ignored. This was rather confusing. The return code is now always ignored and a
comment on install_package() has been added.
Signed-off-by: Pierre Neidhardt <ambrevar at gmail.com>
---
scripts/makepkg.sh.in | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in
index 6e634d9..111f01a 100644
--- a/scripts/makepkg.sh.in
+++ b/scripts/makepkg.sh.in
@@ -2075,6 +2075,7 @@ create_srcpackage() {
rm -rf "${srclinks}"
}
+# this function always returns 0 to make sure clean-up will still occur
install_package() {
(( ! INSTALL )) && return
@@ -2410,7 +2411,7 @@ check_build_status() {
if (( INSTALL )); then
warning "$(gettext "A package has already been built, installing existing package...")"
install_package
- exit $?
+ exit 0
else
error "$(gettext "A package has already been built. (use %s to overwrite)")" "-f"
exit 1
@@ -2433,7 +2434,7 @@ check_build_status() {
if (( INSTALL )); then
warning "$(gettext "The package group has already been built, installing existing packages...")"
install_package
- exit $?
+ exit 0
else
error "$(gettext "The package group has already been built. (use %s to overwrite)")" "-f"
exit 1
@@ -3097,6 +3098,6 @@ msg "$(gettext "Finished making: %s")" "$pkgbase $fullver ($(date))"
install_package
-exit 0 #E_OK
+exit 0
# vim: set noet:
--
1.9.0
More information about the pacman-dev
mailing list