Messages in this thread | | | Date | Sun, 8 Jan 2017 15:24:52 +0100 | From | Jiri Olsa <> | Subject | Re: [PATCH] perf tools: Create libdir directory before installing libperf-jvmti.so |
| |
On Fri, Jan 06, 2017 at 02:18:08PM -0800, Laura Abbott wrote: > The install command for libperf-jvmti.so does not check if libdir exists > before installing. This means that when the install command is run: > > install libperf-jvmti.so '/tmp/test_root/usr/lib64'; > > libperf-jvmti.so will get installed to /usr/lib64 as a file and break further > installation. Fix this by ensuring the directory gets created first. > > See https://bugzilla.redhat.com/show_bug.cgi?id=1410296 > > Fixes: d4dfdf00d43e ("perf jvmti: Plug compilation into perf build") > Signed-off-by: Laura Abbott <labbott@redhat.com> > --- > For full disclosure, Fedora is setting NO_JVMTI but this should still be fixed > to allow it to be turned on in the future.
Acked-by: Jiri Olsa <jolsa@redhat.com>
I have patches for fedora to install that, looks like you guys are not planning to enable it, so I'll post it soon ;-)
thanks, jirka
| |