Re: Various build failures in current bzr tree

[Lennart Sorensen]

On Thu, Feb 09, 2012 at 11:59:58PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko 
wrote:
> On 09.02.2012 20:02, Lennart Sorensen wrote:
> >I am trying to build a powerpc build using the current bzr tree, and
> >have hit a few problems so far.
> >
> >I am trying to use gcc-4.4, which seems to cause issues with -Werror.
> >I think that is gcc's fault, and --disable-werror gets around that,
> >so not a big deal.
> What is the exact warning?

Complains about something like 'type punning breaks strict aliasing
on diskfilter.19, diskfilter.20, etc.  I saw it from the nv raid file.
I can run it again if the exact message is important.

> >I am hitting bug #35452 of course caused by commit r3826, but even after
> >fixing that, commit r38xx does the same thing to another set of files.
> How did you fix it?

Going back to the last commit before that change.  So not really fixed,
just avoided.

> >I also hit that grub_arch_dl_get_tramp_got_size isn't defined.  It appears
> >it was cleaned up for powerpc as being unused recently, but dl.c still
> >says that powerpc has it (along with ia64).  Removing powerpc from:
> I have checked and it's still there and is still used. Please check
> that your checkout is correct and if so provide the full log

Funny, the changelog says it was removed from powerpc about 3 weeks ago
by you:

2012-01-18  Vladimir Serbinenko  <address@hidden>

        * grub-core/kern/powerpc/dl.c (grub_arch_dl_get_tramp_got_size): Remove
        set but not used variable.

> >
> >#if defined (__ia64__) || defined (__powerpc__)
> >   grub_arch_dl_get_tramp_got_size (e,&tramp,&got);
> >
> >seems to fix that.
> >
> >Should I file bug reports, or will someone just fix these?

Actually removing powerpc from here builds successfully and grub is
booting the machine, so it seems to work without it.

-- 
Len Sorensen