On Fri, 2018-09-21 at 18:51 +0300, Victor Kustov wrote:
> [1] https://github.com/gentoo/gentoo/pull/9533#discussion_r211739558
> 
> This patch add parameters to compile and install phases in waf-utils
> eclass. Need to solve [1]
> 
> ---
>  eclass/waf-utils.eclass | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/eclass/waf-utils.eclass b/eclass/waf-utils.eclass
> index 4d02483a927..33515b883b5 100644
> --- a/eclass/waf-utils.eclass
> +++ b/eclass/waf-utils.eclass
> @@ -102,8 +102,8 @@ waf-utils_src_compile() {
>       [[ ${WAF_VERBOSE} == ON ]] && _mywafconfig="--verbose"
>  
>       local jobs="--jobs=$(makeopts_jobs)"
> -     echo "\"${WAF_BINARY}\" build ${_mywafconfig} ${jobs}"
> -     "${WAF_BINARY}" ${_mywafconfig} ${jobs} || die "build failed"
> +     echo "\"${WAF_BINARY}\" $@ build ${_mywafconfig} ${jobs}"
> +     "${WAF_BINARY}" $@ ${_mywafconfig} ${jobs} || die "build failed"

"${@}" i.e. quote it or this is going to spew into a mess.

>  }
>  
>  # @FUNCTION: waf-utils_src_install
> @@ -112,8 +112,8 @@ waf-utils_src_compile() {
>  waf-utils_src_install() {
>       debug-print-function ${FUNCNAME} "$@"
>  
> -     echo "\"${WAF_BINARY}\" --destdir=\"${D}\" install"
> -     "${WAF_BINARY}" --destdir="${D}" install  || die "Make install failed"
> +     echo "\"${WAF_BINARY}\" --destdir=\"${D}\" \"$@\" install"
> +     "${WAF_BINARY}" --destdir="${D}" "$@" install  || die "Make install 
> failed"

Likewise.

>  
>       # Manual document installation
>       einstalldocs

-- 
Best regards,
Michał Górny

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to