[lttng-dev] [PATCH 2/2] Add the CFLAGS for an "out of system" building of the gen-tp example files.

[Jimmy Durand Wesolowski]

2013/8/22 Mathieu Desnoyers <mathieu.desnoyers at efficios.com>

> * Jimmy Durand Wesolowski (jimmy.durand.wesolowski at gmail.com) wrote:
> >  %.o: %.tp %.c %.h
> > -     $(LTTNG_GEN_TP_PATH)lttng-gen-tp -o $@ $<
> > +     CFLAGS="$(AM_CPPFLAGS)" $(LTTNG_GEN_TP_PATH)lttng-gen-tp -o $@ $<
>
> The intent behind the examples is that they are installed in
> /usr/share/doc, and should be built both from within the source tree and
> when installed in the system. This dependency on AM_CPPFLAGS does not
> meet this requirement.
>
Actually, the problem was if you add custom CFLAGS (a custom userspace-rcu
location for example), they will not be set here, and thus, the build fails.

Also, you are mixing CPPFLAGS and CFLAGS here.
>
Yes, indeed, it is messy, I will correct this, and send new patches if you
allow
me to.


Thanks,
>
Thank you.
-- 
Jimmy Durand Wesolowski
OpenWide R&D engineer
OneAccess
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lttng.org/pipermail/lttng-dev/attachments/20130822/6397fe00/attachment.html>