[Buildroot] host-binutils broken?

[Buildroot] host-binutils broken?

[Vincent Stehlé]

Hi,

FYI it seems the following recent commit broke compilation of 
host-binutils (2.22 in my case) when compiling for e.g. 
raspberrypi_defconfig:

   6acb2e1 binutils: backport gas xtensa jump trampolines

If I revert it, host-binutils builds fine.

Best regards,

V.

[Buildroot] host-binutils broken?

[Max Filippov]

Hi Vincent,

On Fri, Apr 4, 2014 at 9:35 PM, Vincent Stehl?
<vincent.stehle@freescale.com> wrote:
> FYI it seems the following recent commit broke compilation of host-binutils
> (2.22 in my case) when compiling for e.g. raspberrypi_defconfig:
>
>   6acb2e1 binutils: backport gas xtensa jump trampolines
>
> If I revert it, host-binutils builds fine.

I couldn't reproduce it, the following steps complete successfully for me:

$ git checkout f90760fe0ed7317510bc6f44c5508e2072ed217d
$ mkdir ../build
$ make O=`pwd`/../build raspberrypi_defconfig
$ make O=`pwd`/../build toolchain

Could you please share your .config and build log?

-- 
Thanks.
-- Max

[Buildroot] host-binutils broken?

[Vincent Stehlé]

Max Filippov <jcmvbkbc@...> writes:
> I couldn't reproduce it,
(..)
> Could you please share your .config and build log?

Hi Max,

Thanks for looking into this issue. I could reproduce it at home in the 
following way:

  $ git clean -fdx
  $ git checkout f90760fe0ed7317510bc6f44c5508e2072ed217d
  $ make raspberrypi_defconfig
  $ make |& tee log

It "chokes" when making as.info, during install-am.

Here is the log: http://pastebin.com/nsUjcZXf
And here is the .config: http://pastebin.com/cyuKb9yA

Best regards,

V.

[Buildroot] host-binutils broken?

[Max Filippov]

On Sat, Apr 5, 2014 at 1:21 AM, Vincent Stehl?
<vincent.stehle@laposte.net> wrote:
> Thanks for looking into this issue. I could reproduce it at home in the
> following way:
>
>   $ git clean -fdx
>   $ git checkout f90760fe0ed7317510bc6f44c5508e2072ed217d
>   $ make raspberrypi_defconfig
>   $ make |& tee log
>
> It "chokes" when making as.info, during install-am.

Doesn't produce even a single warning for me... What's your makeinfo version?

> Here is the log: http://pastebin.com/nsUjcZXf
> And here is the .config: http://pastebin.com/cyuKb9yA

-- 
Thanks.
-- Max

[Buildroot] host-binutils broken?

[Vincent]

On 04/05/2014 12:46 AM, Max Filippov wrote:
(..)
 > Doesn't produce even a single warning for me...

Hi Max,

Thanks for re-trying. You are lucky that it works for you :)

 > What's your makeinfo version?

This is:

   makeinfo (GNU texinfo) 5.2

...on debian testing, from package texinfo 5.2.0.dfsg.1-2.

Does this call for a host-texinfo / host-makeinfo package?

Best regards,

V.

[Buildroot] host-binutils broken?

[Thomas Petazzoni]

Dear Vincent,

On Sat, 05 Apr 2014 14:22:49 +0200, Vincent wrote:

> Thanks for re-trying. You are lucky that it works for you :)
> 
>  > What's your makeinfo version?
> 
> This is:
> 
>    makeinfo (GNU texinfo) 5.2
> 
> ...on debian testing, from package texinfo 5.2.0.dfsg.1-2.
> 
> Does this call for a host-texinfo / host-makeinfo package?

We already have, and in some situations, binutils depends on it:

ifeq ($(BINUTILS_FROM_GIT),y)
BINUTILS_DEPENDENCIES += host-texinfo host-flex host-bison
HOST_BINUTILS_DEPENDENCIES += host-texinfo host-flex host-bison
endif

I believe what's happening is that the patches added by Max are making
certain files newer, which triggers a rebuild of the documentation in
binutils, which normally doesn't happen with release tarballs.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] host-binutils broken?

[Max Filippov]

On Sat, Apr 5, 2014 at 4:22 PM, Vincent <vincent.stehle@laposte.net> wrote:
> On 04/05/2014 12:46 AM, Max Filippov wrote:
>> What's your makeinfo version?
> This is:
>
>   makeinfo (GNU texinfo) 5.2
>
> ...on debian testing, from package texinfo 5.2.0.dfsg.1-2.

Ok, I'll try this version (I currently have 4.13 on build machine).

> Does this call for a host-texinfo / host-makeinfo package?

-- 
Thanks.
-- Max

[Buildroot] host-binutils broken?

[Max Filippov]

On Sat, Apr 5, 2014 at 6:04 PM, Max Filippov <jcmvbkbc@gmail.com> wrote:
> On Sat, Apr 5, 2014 at 4:22 PM, Vincent <vincent.stehle@laposte.net> wrote:
>> On 04/05/2014 12:46 AM, Max Filippov wrote:
>>> What's your makeinfo version?
>> This is:
>>
>>   makeinfo (GNU texinfo) 5.2
>>
>> ...on debian testing, from package texinfo 5.2.0.dfsg.1-2.
>
> Ok, I'll try this version (I currently have 4.13 on build machine).

Yes, now make as.info fails for me too. However, it fails for me both
with and without
my patch. Seems to me the easiest fix would be to drop *.texi hunks from my
patch to binutils to just avoid as.info rebuild. Ok?

-- 
Thanks.
-- Max

[Buildroot] host-binutils broken?

[Thomas Petazzoni]

Dear Max Filippov,

On Sat, 5 Apr 2014 18:35:12 +0400, Max Filippov wrote:

> Yes, now make as.info fails for me too. However, it fails for me both
> with and without
> my patch. Seems to me the easiest fix would be to drop *.texi hunks from my
> patch to binutils to just avoid as.info rebuild. Ok?

Sounds good. Can you send a followup patch that does this?

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com