On Sat, Nov 23, 2013 at 12:27:48AM +0900, Namhyung Kim wrote: > 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
this is where trace-cmd stores them: $(HOME)/.trace-cmd/plugins $(prefix)/$(libdir)/trace-cmd/plugins but I guess we want global place to install plugins, the traceevent by default uses: $(HOME)/.traceevent/plugins $(prefix)/lib/traceevent/plugins How about that? thanks, jirka -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/