Re: [PATCH 02/10] gnu: Add propeller-binutils.

[Ricardo Wurmus]

Ludovic Courtès <address@hidden> writes:

> Ricardo Wurmus <address@hidden> skribis:
>
>> * gnu/packages/embedded.scm (propeller-binutils): New variable.
>
> [...]
>
>> +         ,@(substitute-keyword-arguments (package-arguments xbinutils)
>> +             ((#:configure-flags flags)
>> +              `(cons "--disable-nls" ,flags)))))
>
> That shouldn't be needed.

Okay.

>> +      (native-inputs
>> +       `(("bison" ,bison)
>> +         ("flex" ,flex)
>> +         ("texinfo" ,texinfo)
>> +         ("dejagnu" ,dejagnu)
>
> The test failures you see are because you added DejaGNU here.  The other
> Binutils instances don’t have it, so few tests are run I guess.  :-)
>
> If this is not needed for bootstrapping, I’d suggest removing it.

I added DejaGNU because it provides “runtest”, which is needed by the
tests.  Without DejaGNU the check phase fails trying to execute
“runtest”.

--
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
http://elephly.net