Re: [regressions] Re: kbuild: save ARCH & CROSS_COMPILE when building a kernel

[Ingo Molnar <mingo@elte.hu>]

* Sam Ravnborg <sam@ravnborg.org> wrote:

> On Fri, Oct 09, 2009 at 11:26:48AM +0200, Ingo Molnar wrote:
> > 
> > * Sam Ravnborg <sam@ravnborg.org> wrote:
> > 
> > > On Sat, Oct 03, 2009 at 12:34:41PM +0200, Ingo Molnar wrote:
> > > > 
> > > > Sam,
> > > > 
> > > > that arch-cache thing introduced in:
> > > > 
> > > >  5755433: kbuild: save ARCH & CROSS_COMPILE when building a kernel
> > > > 
> > > > isnt working very well in my experience.
> > > > 
> > > > 1) it's a nuisance in cross-builds (it broke several cross-build scripts 
> > > >    of mine)
> > >
> > > On the contrary it has helped me big time on my cross builds. [...]
> > 
> > Could you please try to do that without hurting the existing scripting 
> > and existing workflow of other people?
> 
> The only message in the above was that this was also beneficial
> to some - and not a nusiance only.
> 
> > 
> > > No need to script is all to remember to set the correct ARCH + 
> > > CROSS_COMPILE settings.
> > 
> > Then put your new state into the .config instead of this stupid 
> > include/generated/ extra state that is a _big_ regression over v2.6.31 
> > in terms of cross-build usability.
> 
> I cannot see why saving this in include/generated/*
> has anything to do with your complains. I guess you did not look
> at the patch in question.
> 
> What is hurting you and several other users are _not_ related
> to the fact that we save ARCH and CROSS_COMPILE setting or where
> we save it.
> What hurt you is the consistency checks that was added
> to try to catch people in doing something unexpected.
> 
> Unexpected like trying to build a 64 bit kernel where they
> previosuly were building a 32 bit kernel.
> Or when they suddenly used a different set of binutils
> because they changed CROSS_COMPILE settings.
> 
> 
> The patch in question is reverted in my next push to Linus.

Thanks! Mind sending the fix/revert here too, so that we can test 
whether that resolves all the problems?

	Ingo