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

[Namhyung Kim <namhyung@kernel.org>]

On Wed, 17 Oct 2012 14:19:45 -0300, Arnaldo Carvalho de Melo wrote:
> From: Arnaldo Carvalho de Melo <acme@redhat.com>
>
> The evsel methods to read tracepoint fields uses libtraceevent
> functions, becoming needed by the python binding as well.

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

The libtraceevent.a will be in
python_ext_build/tmp/../../../lib/traceevent/libtraceevent.a if make was
invoked without the O.

Makefile contains following:

  TRACE_EVENT_DIR = ../lib/traceevent/
  
  ifneq ($(OUTPUT),)
  	TE_PATH=$(OUTPUT)
  else
  	TE_PATH=$(TRACE_EVENT_DIR)
  endif
  
  LIBTRACEEVENT = $(TE_PATH)libtraceevent.a
  TE_LIB := -L$(TE_PATH) -ltraceevent

Thanks,
Namhyung

>
> Cc: David Ahern <dsahern@gmail.com>
> Cc: Frederic Weisbecker <fweisbec@gmail.com>
> Cc: Jiri Olsa <jolsa@redhat.com>
> Cc: Mike Galbraith <efault@gmx.de>
> Cc: Namhyung Kim <namhyung@gmail.com>
> Cc: Paul Mackerras <paulus@samba.org>
> Cc: Peter Zijlstra <peterz@infradead.org>
> Cc: Stephane Eranian <eranian@google.com>
> Link: http://lkml.kernel.org/n/tip-j3o4v7jyvp9ke9n230l96a1m@git.kernel.org
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> ---
>  tools/perf/util/setup.py |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/tools/perf/util/setup.py b/tools/perf/util/setup.py
> index d0f9f29..09c3cea 100644
> --- a/tools/perf/util/setup.py
> +++ b/tools/perf/util/setup.py
> @@ -31,6 +31,7 @@ perf = Extension('perf',
>  		  sources = ext_sources,
>  		  include_dirs = ['util/include'],
>  		  extra_compile_args = cflags,
> +		  extra_objects = [build_tmp + '/../../libtraceevent.a'],
>                   )
>  
>  setup(name='perf',