[PATCH 2/3] buildsys: fix handling of CFLAGS-OMIT from global CFLAGS

[Florian Fainelli]

Hello,

2013/9/13 Filippo ARCIDIACONO <filippo.arcidiacono at st.com>:
> Currently the Makefile rules are not removing from the global CFLAGS
> any of the OMIT cflags declared at file level. This patch fixes it.

Do we really need to have two different ways of achieving the same
thing? It seems to me that doing:

CFLAGS_foo.c += -fno-bar

is equivalent to:

CFLAGS-OMIT_foo.c += -fbar

I prefer the first one because we do not need to introduce another
infrastructure than the one already existing.

>
> Signed-off-by: Carmelo Amoroso <carmelo.amoroso at st.com>
> Signed-off-by: Filippo Arcidiacono <filippo.arcidiacono at st.com>
> ---
>  Makerules |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/Makerules b/Makerules
> index 367daa5..54344c5 100644
> --- a/Makerules
> +++ b/Makerules
> @@ -230,7 +230,8 @@ collect_multi_flags = $(CFLAGS-$(notdir $(patsubst %/,%,$(dir $(d)))))
>
>  CFLAGS_gen.dep = -MT $@ -MD -MP -MF $(dir $@).$(notdir $@).dep
>
> -cmd_compile.c = $(CC) -c $< -o $@ $(CFLAGS) \
> +cmd_compile.c = $(CC) -c $< -o $@ \
> +       $(filter-out $(CFLAGS-OMIT-$(notdir $<)),$(CFLAGS)) \
>         $(CFLAGS-$(suffix $@)) \
>         $(filter-out $(CFLAGS-OMIT-$(notdir $<)),$(CFLAGS-$(notdir $(<D)))) \
>         $(CFLAGS-$(subst $(top_srcdir),,$(dir $<))) \
> --
> 1.7.7.6
>
> _______________________________________________
> uClibc mailing list
> uClibc at uclibc.org
> http://lists.busybox.net/mailman/listinfo/uclibc



-- 
Florian