Re: [PATCH 0/3] perf tools: Plug jvmti compilation into perf build

[Stephane Eranian]

On Fri, Nov 11, 2016 at 1:51 AM, Jiri Olsa <jolsa@xxxxxxxxxx> wrote:
>
> On Wed, Nov 02, 2016 at 02:35:46PM +0100, Jiri Olsa wrote:
> > hi,
> > adding jvmti compilation under perf build, so it's easy
> > to put it under rpm.
> >
> > I plan on updating fedora/rhel perf rpm, to have the jvmti
> > placed in like:
> >
> >   $ rpm -qpl build/x86_64/perf-4.8.5-300.fc24.x86_64.rpm | grep jvmti
> >   /usr/lib64/libperf-jvmti.so
> >
> > Stephane,
> > I'm keeping the jvmti/Makefile as I dont fully follow
> > the need for .lo objects. Please let me know if you'd
> > be ok with this new way of building jvmti lib.
>
> ping

Sorry for the delay. Will look at this today and get back to you.

>
>
>
> thanks,
> jirka
>
> >
> > Also available in:
> >   git://git.kernel.org/pub/scm/linux/kernel/git/jolsa/perf.git
> >   perf/fixes
> >
> > thanks,
> > jirla
> >
> >
> > ---
> > Jiri Olsa (3):
> >       tools build: Add CFLAGS_REMOVE_* support
> >       tools build: Add jvmti feature detection support
> >       perf tools: Plug jvmti compilation into perf build
> >
> >  tools/build/Build.include           |  4 +++-
> >  tools/build/Documentation/Build.txt |  6 ++++--
> >  tools/build/feature/Makefile        |  6 +++++-
> >  tools/build/feature/test-jvmti.c    | 13 +++++++++++++
> >  tools/perf/Makefile.config          | 26 ++++++++++++++++++++++++++
> >  tools/perf/Makefile.perf            | 24 +++++++++++++++++++++++-
> >  tools/perf/jvmti/Build              |  8 ++++++++
> >  tools/perf/tests/make               |  2 +-
> >  8 files changed, 83 insertions(+), 6 deletions(-)
> >  create mode 100644 tools/build/feature/test-jvmti.c
> >  create mode 100644 tools/perf/jvmti/Build