[U-Boot-Users] Broken (ARM) build, was: u-boot patch submission process

[Dirk Behme]

Haavard Skinnemoen wrote:
> On Mon, 11 Feb 2008 05:43:18 +0100
> Dirk Behme <dirk.behme at googlemail.com> wrote:
> 
> 
>>[U-Boot-Users] [PATCH] [DRIVERS] <asm/mpc8xxx_spi.h> is onlyavailable
>>for ppc boards.
>>
>>I'm assuming this will eventually be properly patched in the main tree
>>- meanwhile anyone building ARM code can apply this patch
>>-- cut --
>>
>>This is quite annoying, particularly if you should test patches 
>>already in that tree or against that. IMHO the arm.git should contain 
>>the fix temporarily to be able to build cleanly instead of staying 
>>with a broken tree.
> 
> 
> You're damn right it's annoying, but applying the fix to every single
> arch tree just isn't the right solution. That will cause mayhem when it
> turns out that every arch maintainer picked up a different fix among
> the 6 or so that were posted.
> 
> But if Wolfgang doesn't wake up and apply a fix pretty soon, I guess
> that might be exactly what's going to happen.
> 
> 
>>To workaround I would propose that you test MAKEALL against mainline 
>>U-Boot git.
> 
> Won't help since it's mainline that is broken. 

Uups, yes. Last time I tried mainline it worked, so I thought only 
arm.git has this issue.

But testing arm.git and mainline git from today, yes, *both* stop with

mpc8xxx_spi.c:26:29: error: asm/mpc8xxx_spi.h: No such file or directory

At least for (ARM based) davinci_dvevm_config I tried.

Sorry for the confusion.

> And it's not only ARM --
> every single architecture except PPC has been broken for several weeks.

Seems that we should asap have a fix for this.

Dirk

> The first fix was posted less than three hours after the breakage was
> introduced, but no fix has yet been merged.
> 
> I'm starting to get seriously pissed off about this whole situation and
> how it's been handled. Wolfgang, you can't with a straight face demand
> that people always run the latest git snapshot WHEN IT DOESN'T EVEN
> BUILD AND HAS BEEN THAT WAY FOR SEVERAL WEEKS!
> 
> Haavard
>