[lttng-dev] [LTTNG][LTTng-ust] "Out of system" building patches.

[Mathieu Desnoyers]

* Mathieu Desnoyers (mathieu.desnoyers at efficios.com) wrote:
> * Jimmy Durand Wesolowski (jimmy.durand.wesolowski at gmail.com) wrote:
> > 2013/8/26 Mathieu Desnoyers <mathieu.desnoyers at efficios.com>
> > 
> > > Now I just have to figure out how to teach lttng-gen-tp about CPPFLAGS
> > > :-/
> > >
> > 
> > My patch is correcting this, and the new version does take your
> > comment into account.
> 
> Actually, I just pushed a fix for this into master (I had to test all
> this myself to understand what happens):
> 
> commit 9ec23f2e52b3b0bd8b80afbe2e8e8ae1ab222c6b
> Author: Mathieu Desnoyers <mathieu.desnoyers at efficios.com>
> Date:   Mon Aug 26 11:14:56 2013 -0400
> 
>     Fix: doc/examples cross-build
>     
>     We need to pass automake's CPPFLAGS and LDFLAGS to examples. Add CFLAGS
>     too for completeness.
>     
>     Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers at efficios.com>
> 
> We still have issues with lttng-gen-tp though: I'm looking for a way to
> change its Makefile so it adds "AM_CPPFLAGS" to the CFLAGS passed to the
> compiler. Ideas ?

I got it to work. See those commits in master:

commit 256175cffb00b15543bd49a4268e6c528bd51fce
Author: Mathieu Desnoyers <mathieu.desnoyers at efficios.com>
Date:   Mon Aug 26 11:26:09 2013 -0400

    doc/examples/gen-tp: pass automake CPPFLAGS/CFLAGS/LDFLAGS
    
    Fix cross-build.
    
    Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers at efficios.com>

commit 170423b056ecdef441820a54be350605e4dc8951
Author: Mathieu Desnoyers <mathieu.desnoyers at efficios.com>
Date:   Mon Aug 26 11:24:26 2013 -0400

    tools/lttng-gen-tp: honor CPPFLAGS and LDFLAGS
    
    Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers at efficios.com>

Thanks,

Mathieu

-- 
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com