[U-Boot] v2015.01-rc4 REGRESSION: "scripts: fix binutils-version.sh" breaks things for non Linaro toolchains

[Tom Rini]

On Tue, Jan 06, 2015 at 11:27:43AM +0100, Hans de Goede wrote:
> Hi all,
> 
> ping ? current master still has this regression, it is not fatal, but it is not
> pretty either.

Did you see my earlier reply?  It's OK with vanilla toolchains (see
ELDK) and Linaro ones, but breaking on how Fedora mangles the version
string.  I'm _not_ saying it's a Fedora problem, but can you poke at
this a bit?  If not, I'll play with echo and see if I can do it.
Thanks!

> 
> Regards,
> 
> Hans
> 
> 
> On 30-12-14 11:55, Hans de Goede wrote:
> >Hi,
> >
> >I noticed $subject while doing a MAKEALL.
> >
> >It seems that this commit:
> >http://git.denx.de/?p=u-boot.git;a=commit;h=73c25753060c58e4c339fba306ed0ded0c335748
> >
> >Breaks things with Fedora's arm toolchain:
> >
> >[hans at shalem u-boot]$ scripts/binutils-version.sh arm-linux-gnu-as
> >scripts/binutils-version.sh: line 22: printf: GNU: invalid number
> >scripts/binutils-version.sh: line 22: printf: assembler: invalid number
> >0000
> >scripts/binutils-version.sh: line 22: printf: version: invalid number
> >0002
> >2400
> >
> >And:
> >
> >[hans at shalem ~]$ arm-linux-gnu-as --version
> >GNU assembler version 2.24.0-6.fc21 20140613
> >Copyright 2013 Free Software Foundation, Inc.
> >This program is free software; you may redistribute it under the terms of
> >the GNU General Public License version 3 or later.
> >This program has absolutely no warranty.
> >This assembler was configured for a target of `arm-linux-gnueabi'.
> >
> >Which makes $version_string: "GNU assembler version 2.24.0-6.fc21 20140613"
> >and $MAJOR: "GNU assembler version 2"
> >
> >Regards,
> >
> >Hans
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150106/d543b9f8/attachment.pgp>