[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!
- [PATCH 01/10] gnu: Add dummy linker for propeller-elf., Ricardo Wurmus, 2016/12/26
- [PATCH 02/10] gnu: Add propeller-binutils., Ricardo Wurmus, 2016/12/26
- Re: [PATCH 02/10] gnu: Add propeller-binutils.,
Ludovic Courtès <=
- [PATCH 03/10] gnu: Add propeller-gcc., Ricardo Wurmus, 2016/12/26
- [PATCH 04/10] gnu: Add proplib., Ricardo Wurmus, 2016/12/26
- [PATCH 05/10] gnu: Add propeller-toolchain., Ricardo Wurmus, 2016/12/26
- [PATCH 06/10] gnu: Add openspin., Ricardo Wurmus, 2016/12/26
- [PATCH 07/10] gnu: Add propeller-load., Ricardo Wurmus, 2016/12/26