[PATCH] kexec-tools: don't include every archs Makefile
Michael Neuling
mikey at neuling.org
Wed May 5 00:00:58 EDT 2010
In message <20100505033123.GB2612 at verge.net.au> you wrote:
> On Wed, May 05, 2010 at 11:51:10AM +1000, Michael Neuling wrote:
> > This patches changes the kexec/Makefile to only include the required
> > arch Makefile.
> >
> > Including all arch Makefiles causes unexpected results as changing one
> > archs Makefile effects all other archs.
> >
> > ppc64 recently broke surprisingly because
> > 6adc05c6e3fdbc8b9f5d915af78ca05d0a09cb17 "some kexec MIPS
> > improvements" added "CFLAGS +=-Wall -Werror" to
> > kexec/arch/mips/Makefile. This shouldn't happen.
> >
> > Signed-off-by: Michael Neuling <mikey at neuling.org>
> > ---
> > Horms: Tested by compiling on x86 and ppc64, but probably needs
> > testing on other archs before being committed.
>
> Unfortunately this breaks make tarball - only kexec/arch/$ARCH is
> included in the tarball.
I figured it might break something :-)
BTW make tarball works here.
> Perhaps we could introduce $(ARCH)_FLAGS, which I think is
> consistent with the handling of other variables by the kexec build.
Seems like a bit of a band aid.
It still seems pretty broken that we use arch/mips/Makefile for ppc64.
Looking at 6adc05c6e3fdbc8b9f5d915af78ca05d0a09cb17 and trying to work
out how it broke ppc64 was pretty annoying.
Mikey
More information about the kexec
mailing list