guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Ludovic Courtès
Subject: Re: [PATCH 02/10] gnu: Add propeller-binutils.
Date: Sat, 31 Dec 2016 01:31:36 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

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.

> +      (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.

Otherwise LGTM, thanks!



reply via email to

[Prev in Thread] Current Thread [Next in Thread]