Merge pull request #1442 from Tagadda/patch-1

fix ynh_exec_*: ensure the arg message is used
This commit is contained in:
Alexandre Aubin 2022-02-27 14:21:12 +01:00 committed by GitHub
commit 680811eeff
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -95,10 +95,10 @@ ynh_exec_err() {
# we detect this by checking that there's no 2nd arg, and $1 contains a space
if [[ "$#" -eq 1 ]] && [[ "$1" == *" "* ]]
then
ynh_print_err "$(eval $@)"
ynh_print_err --message="$(eval $@)"
else
# Note that "$@" is used and not $@, c.f. https://unix.stackexchange.com/a/129077
ynh_print_err "$("$@")"
ynh_print_err --message="$("$@")"
fi
}
@ -116,10 +116,10 @@ ynh_exec_warn() {
# we detect this by checking that there's no 2nd arg, and $1 contains a space
if [[ "$#" -eq 1 ]] && [[ "$1" == *" "* ]]
then
ynh_print_warn "$(eval $@)"
ynh_print_warn --message="$(eval $@)"
else
# Note that "$@" is used and not $@, c.f. https://unix.stackexchange.com/a/129077
ynh_print_warn "$("$@")"
ynh_print_warn --message="$("$@")"
fi
}