[Buildroot] [PATCH 1/1] httping: new package
Arnout Vandecappelle
arnout at mind.be
Sat Jan 19 13:51:19 UTC 2013
On 14/01/13 14:02, Peter Korsgaard wrote:
>>>>>> "Gilles" == Gilles Talis<gilles.talis at gmail.com> writes:
>
> Gilles> I forgot to copy mailing list.
> Gilles> Sorry about that and thanks for your help.
> Gilles> Gilles.
>
> Gilles> 2013/1/13 Gilles Talis<gilles.talis at gmail.com>
>
> Gilles> Dear Thomas,
>
> Gilles> I thought I knew Makefiles better than that. Well, I was wrong :-).
> Gilles> The reason why TARGET_CONFIGURE_OPTS does not work in this case is because
> Gilles> it overrides the original Makefile's CFLAGS variable (that should be kept
> Gilles> in order for target to compile). Apart from adding "override" in the
> Gilles> package Makefile, I actually do not see how to use TARGET_CONFIGURE_OPTS
> Gilles> and still keep Makefile CFLAGS. Any hint?
> Gilles> This will surely help me for future patches/projects.
>
> That's indeed how it is normally done.
>
> override CFLAGS += ..
No.
Normally CFLAGS should be assigned to either as
CFLAGS +=
or
CFLAGS ?=
and the user-settable CFLAGS are passed in the environment.
I think most Makefiles follow that convention.
Patch follows.
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
More information about the buildroot
mailing list