<div dir="ltr">2013/8/22 Mathieu Desnoyers <span dir="ltr"><<a href="mailto:mathieu.desnoyers@efficios.com" target="_blank">mathieu.desnoyers@efficios.com</a>></span><br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">* Jimmy Durand Wesolowski (<a href="mailto:jimmy.durand.wesolowski@gmail.com">jimmy.durand.wesolowski@gmail.com</a>) wrote:<br>

> 2013/8/22 Mathieu Desnoyers <<a href="mailto:mathieu.desnoyers@efficios.com">mathieu.desnoyers@efficios.com</a>><br>
><br>
> > * Jimmy Durand Wesolowski (<a href="mailto:jimmy.durand.wesolowski@gmail.com">jimmy.durand.wesolowski@gmail.com</a>) wrote:<br>
> > >  %.o: %.tp %.c %.h<br>
> > > -     $(LTTNG_GEN_TP_PATH)lttng-gen-tp -o $@ $<<br>
> > > +     CFLAGS="$(AM_CPPFLAGS)" $(LTTNG_GEN_TP_PATH)lttng-gen-tp -o $@ $<<br>
> ><br>
> > The intent behind the examples is that they are installed in<br>
> > /usr/share/doc, and should be built both from within the source tree and<br>
> > when installed in the system. This dependency on AM_CPPFLAGS does not<br>
> > meet this requirement.<br>
> ><br>
> Actually, the problem was if you add custom CFLAGS (a custom userspace-rcu<br>
> location for example), they will not be set here, and thus, the build fails.<br>
<br>
</div>I understand that we might need to fix something, but I think this is<br>
not done at the right level in your patches.<br>
<br>
We could keep the CFLAGS/LDFLAGS/CPPFLAGS use as we have currently in<br>
the tree, but also modify doc/examples/Makefile.am so it passes extra<br>
flags (the AM_CFLAGS/AM_LDFLAGS/AM_CPPFLAGS) to the subdirectories when<br>
invoking their respective makefile.<br>
<br>
Therefore, it would not add automake-specific stuff into a Makefile that<br>
is meant to be installed in /usr/share/doc.<br></blockquote><div>Oh, OK.</div><div>Do you want me to add an</div><div>export CFLAGS</div><div>export CPPFLAGS</div><div>export CC</div><div>in the doc/examples/Makefile.am ?</div>
<div>Or add it on the make calling loop if you prefer avoiding exports ?<br></div><div><br></div><div>This would allow no AM_* reference in a generic Makefile.</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Thanks,<br></blockquote><div>Thank you.</div></div>-- <br><div dir="ltr">Jimmy Durand Wesolowski<br><div>OpenWide R&D engineer<br></div><div>OneAccess</div></div>
</div></div>