Am 30.06.2010 15:09, schrieb Allan McRae:
> When makepkg catches a command failure it currently prints "Aborting..."
> and exits.  Clarify the reason for aborting from packaging is a failure
> during one of the PKGBUILDs functions.
> 
> Signed-off-by: Allan McRae <al...@archlinux.org>

Signed-off-by: Thomas Bächler <tho...@archlinux.org>

I was very confused when I saw that 'plain' error message for the first
time, thought maybe makepkg had crashed or something.

This is much better and way less scary (as it includes the fancy colored
==>).

> ---
> 
> The error now looks like:
> ==> ERROR: A failure occurred in package().
>     Aborting...
> 
>  scripts/makepkg.sh.in |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in
> index cbc344d..fdedc89 100644
> --- a/scripts/makepkg.sh.in
> +++ b/scripts/makepkg.sh.in
> @@ -721,6 +721,7 @@ error_function() {
>       fi
>       # first exit all subshells, then print the error
>       if (( ! BASH_SUBSHELL )); then
> +             error "$(gettext "A failure occurred in %s().")" "$1"
>               plain "$(gettext "Aborting...")"
>               remove_deps
>       fi
> @@ -768,7 +769,7 @@ run_function() {
>               tee "$BUILDLOG" < "$logpipe" &
>               exec 1>"$logpipe" 2>"$logpipe"
>               restoretrap=$(trap -p ERR)
> -             trap 'error_function' ERR
> +             trap 'error_function $pkgfunc' ERR
>               $pkgfunc 2>&1
>               eval $restoretrap
>               sync
> @@ -776,7 +777,7 @@ run_function() {
>               rm "$logpipe"
>       else
>               restoretrap=$(trap -p ERR)
> -             trap 'error_function' ERR
> +             trap 'error_function $pkgfunc' ERR
>               $pkgfunc 2>&1
>               eval $restoretrap
>       fi


Attachment: signature.asc
Description: OpenPGP digital signature



Reply via email to