On Sun, Apr 17, 2016 at 10:50 AM, Ian Stakenvicius <a...@gentoo.org> wrote: > >> On Apr 17, 2016, at 10:45 AM, Mike Gilbert <flop...@gentoo.org> wrote: >> >>> On Sun, Apr 17, 2016 at 10:31 AM, Ian Stakenvicius <a...@gentoo.org> wrote: >>> >>> >>> Sent from an iPhone, sorry for the HTML... >>> >>>> On Apr 17, 2016, at 10:13 AM, Mike Gilbert <flop...@gentoo.org> wrote: >>>> >>>> @@ -487,10 +497,17 @@ xorg-2_src_configure() { >>>> xorg-2_src_compile() { >>>> debug-print-function ${FUNCNAME} "$@" >>>> >>>> + local makeargs=( "$@" ) >>>> + >>>> if [[ ${XORG_MULTILIB} == yes ]]; then >>>> - autotools-multilib_src_compile "$@" >>>> + if ! declare -f multilib_src_compile >/dev/null; then >>>> + multilib_src_compile() { >>>> + emake "${makeargs[@]}" >>>> + } >>>> + fi >>>> + multilib-minimal_src_compile >>>> else >>>> - autotools-utils_src_compile "$@" >>>> + emake "${makeargs[@]}" >>>> fi >>>> } >>> >>> Does the src_compile code need to be as complex as this? Why not just a >>> 'if $XORG_MULTILIB ; then multilib-minimal_src_compile "$@"; else emake >>> "$@"; fi '? >>> >>> ...or even 'default' instead of emake...? >> >> multilib-mininmal_src_compile and default_src_comple do not provide >> any method to pass arguments to emake. If I recall correctly, there is >> at least one ebuild that needs to do so. >> > > That would do it then... > > mgorny how difficult do you think it would be to pass extra "$@" bits through > to the default multilib_src_compile? Or does that just call 'default' too... > (can't check right now) >
If multilib_src_compile is undefined, mutlilib-minimal_src_compile calls default_src_compile.