Re: [PATCH 1/5] gnu: Add avr-binutils.

[Manolis Ragkousis <manolis837@gmail.com>]

Hey Ludo,

On 04/14/16 20:04, Ludovic Courtès wrote:

> AFAICS --target=avr is redundant:
> 
> --8<---------------cut here---------------start------------->8---
> $ git describe
> v0.10.0-204-ga2d0e20
> $ wget -q -O - $(./pre-inst-env guix build --log-file -e '((@ (gnu packages cross-base) cross-binutils) "avr")') |grep -e --target
> configure flags: ("CONFIG_SHELL=/gnu/store/b1yqjimbdh5bf9jnizd4h7yf110744j2-bash-4.3.42/bin/bash" "SHELL=/gnu/store/b1yqjimbdh5bf9jnizd4h7yf110744j2-bash-4.3.42/bin/bash" "--prefix=/gnu/store/gqmq8gis9igpg4xy6022mvq8qpvp7kk4-binutils-cross-avr-2.25.1" "--enable-fast-install" "--build=x86_64-unknown-linux-gnu" "--target=avr" "--with-sysroot=/" "LDFLAGS=-static-libgcc" "--with-lib-path=/no-ld-lib-path" "--disable-werror" "--enable-install-libbfd" "--enable-deterministic-archives")
> --8<---------------cut here---------------end--------------->8---
> 
> … and --disable-nls is orthogonal and probably unnecessary.
> 
> Can we skip this patch?  :-)

We added --disable-nls mostly because it was suggested from the avr-libc
manual to be passed to binutils and gcc.
<http://www.nongnu.org/avr-libc/user-manual/install_tools.html>

David can you build the toolchain without it and see if it works?

And I would really prefer if we had a way to explicitly install
avr-binutils.  This way it will be more clear to someone only wanting to
install this specific package.

We could keep something like this

(define-public avr-binutils
  (package
    (inherit (cross-binutils "avr"))
    (name "avr-binutils")))

Manolis