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.

Reply via email to