Re: [PATCH 09/11] perf python: Link with libtraceevent

[Ingo Molnar <mingo@kernel.org>]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=unknown-8bit, Size: 1771 bytes --]


* Namhyung Kim <namhyung@kernel.org> wrote:

> 2012-10-18 (???), 11:38 -0300, Arnaldo Carvalho de Melo:
> > Em Thu, Oct 18, 2012 at 11:00:33AM -0300, Arnaldo Carvalho de Melo escreveu:
> > > Em Thu, Oct 18, 2012 at 10:38:18AM +0900, Namhyung Kim escreveu:
> > > > On Wed, 17 Oct 2012 14:19:45 -0300, Arnaldo Carvalho de Melo wrote:
> > > > I got a build error after adding 'extern unsigned int page_size' to
> > > > util/python.c - please see my previous reply.
> > > > 
> > > >   namhyung@sejong:perf$ make
> > > >       SUBDIR ../lib/traceevent/
> > > >       LINK perf
> > > >       GEN python/perf.so
> > > >   gcc: error: python_ext_build/tmp//../../libtraceevent.a: No such file or directory
> > > >   error: command 'gcc' failed with exit status 1
> > > >   cp: cannot stat `python_ext_build/lib/perf.so': No such file or directory
> > > >   make: *** [python/perf.so] Error 1
> > > >   TRACE_EVENT_DIR = ../lib/traceevent/
> >    
> > > >   ifneq ($(OUTPUT),)
> > > >   	TE_PATH=$(OUTPUT)
> > > >   else
> > > >   	TE_PATH=$(TRACE_EVENT_DIR)
> > > >   endif
> > > >   
> > > >   LIBTRACEEVENT = $(TE_PATH)libtraceevent.a
> > > 
> > > I see, so we need to somehow propagate this TE_PATH variable to the setup.py file...
> > 
> > Can you try with the attached patch? I tested it here and works with:
> > 
> >  make -C tools/perf
> > 
> > and with O=
> 
> It works well for me too:
> 
> Tested-by: Namhyung Kim <namhyung@kernel.org>

Even with that I'm getting:

util/python.c: In function ‘initperf’:
util/python.c:1018:2: error: ‘page_size’ undeclared 
(first use in this function)
util/python.c:1018:2: note: each undeclared identifier is 
reported only once for each function it appears in
error: command 'gcc' failed with exit status 1

Thanks,

	Ingo