Re: [PATCH 17/22] perf tools: Add build and install plugins targets

From: Namhyung Kim
Date: Fri Nov 22 2013 - 10:28:01 EST


2013-11-21 (ë), 12:01 +0100, Jiri Olsa:
> Adding 'plugins' target along with the libtraceevent.a,
> so plugins are built together with traceevent library.
>
> Adding 'install-traceevent-plugins' Makefile install target,
> instructing perf to install plugins into:
> $(HOME)/libexec/perf-core/traceevent/plugins
> - if installed localy under $HOME
> $(DESTDIR)/$(prefix)/$(perfexecdir)/traceevent/plugins
> - if installed globally

[SNIP]
> --- a/tools/perf/config/Makefile
> +++ b/tools/perf/config/Makefile
> @@ -578,6 +578,7 @@ else
> lib = lib
> endif
> libdir = $(prefix)/$(lib)
> +plugindir=$(perfexecdir)/traceevent/plugins

Hmm.. it puts traceevent plugins under the perf directory. But I don't
think it's a good idea since libtraceevent can be used independently.

What about something like this?

plugindir = $(perfexecdir)/../traceevent/plugins

Thanks,
Namhyung


--
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/