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

From: Ingo Molnar
Date: Fri Oct 19 2012 - 20:36:20 EST



* Namhyung Kim <namhyung@xxxxxxxxxx> 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@xxxxxxxxxx>

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/