<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="HOEnZb"><div class="h5">* 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>
</div></div>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></blockquote><div>Actually, the problem was if you add custom CFLAGS (a custom userspace-rcu</div><div>location for example), they will not be set here, and thus, the build fails.</div><div><br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Also, you are mixing CPPFLAGS and CFLAGS here.<br></blockquote><div>Yes, indeed, it is messy, I will correct this, and send new patches if you allow</div><div>me to. </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>