Re: [PATCH] perf tools: Fix traceevent plugin path definitions

2014-01-26 Thread Namhyung Kim
On Wed, 22 Jan 2014 10:01:48 -0500, Josh Boyer wrote: > The plugindir_SQ definition contains $(prefix) which is not needed > as the $(libdir) definition already contains prefix in it. This > leads to the path including an extra prefix in it, e.g. /usr/usr/lib64. > > The -DPLUGIN_DIR defintion incl

Re: [PATCH] perf tools: Fix traceevent plugin path definitions

2014-01-22 Thread Jiri Olsa
On Wed, Jan 22, 2014 at 10:01:48AM -0500, Josh Boyer wrote: > The plugindir_SQ definition contains $(prefix) which is not needed > as the $(libdir) definition already contains prefix in it. This > leads to the path including an extra prefix in it, e.g. /usr/usr/lib64. > > The -DPLUGIN_DIR definti

[PATCH] perf tools: Fix traceevent plugin path definitions

2014-01-22 Thread Josh Boyer
The plugindir_SQ definition contains $(prefix) which is not needed as the $(libdir) definition already contains prefix in it. This leads to the path including an extra prefix in it, e.g. /usr/usr/lib64. The -DPLUGIN_DIR defintion includes DESTDIR. This is incorrect, as it sets the plugin search