Re: [PATCH v2 0/3] update of various kbuild flags to allow usersettings

From: Sam Ravnborg
Date: Wed Jul 28 2010 - 14:30:32 EST


On Wed, Jul 28, 2010 at 08:25:23PM +0200, Sam Ravnborg wrote:
> Denys Vlasenko complained that he could not set
> a moduel specific linker flags on the command line.
>
> It turned out that the cause of this was kbuild
> that assigned a value to a variables that
> may be modified by the user.
> So any user supplied value would overwrite
> the kbuild assigned value.
>
> Fix this so we use internal variables for kbuild
> feering up the official variables for use by
> the user.
>
> Changes in v2:
> Fix so we can now specify both module and built-in
> options on command line.
> Combine all *_MODULE changes in one patch.
> Add a new patch for *_KERNEL changes.
> Document the new variables in makefiles.txt
> Document the variables that may be set on the command line
> in kbuild.txt
>
> The frv Makefile saw a bigger cleanup this time.
> I suggest to merge it through kbuild.git because the
> update is dependent on the first patch in this serie.

Forgot the stats:


Sam Ravnborg (3):
kbuild: allow assignment to {A,C,LD}FLAGS_MODULE on the command line
kbuild: allow assignment to {A,C}FLAGS_KERNEL on the command line
frv: clean up arch/frv/Makefile

Documentation/kbuild/kbuild.txt | 26 ++++++++++++++++++++++++--
Documentation/kbuild/makefiles.txt | 27 ++++++++++++++++++++++-----
Makefile | 17 ++++++++++++-----
arch/avr32/Makefile | 2 +-
arch/blackfin/Makefile | 4 ++--
arch/frv/Makefile | 22 ++--------------------
arch/ia64/Makefile | 6 +++---
arch/m32r/Makefile | 4 ++--
arch/m68k/Makefile | 2 +-
arch/mips/Makefile | 6 ++++--
arch/powerpc/Makefile | 2 +-
arch/s390/Makefile | 3 ++-
arch/score/Makefile | 3 ++-
scripts/Makefile.build | 11 +++++++----
scripts/Makefile.modpost | 7 ++++---
15 files changed, 89 insertions(+), 53 deletions(-)

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/