2013/8/22 Mathieu Desnoyers <[email protected]>

> * Jimmy Durand Wesolowski ([email protected]) 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
_______________________________________________
lttng-dev mailing list
[email protected]
http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Reply via email to