Message ID | 20191114133719.309-1-sudipm.mukherjee@gmail.com (mailing list archive) |
---|---|
State | Accepted |
Headers | show |
Series | libtraceevent: fix header installation | expand |
Arnaldo, Can you take this? Reviewed-by: Steven Rostedt (VMware) <rostedt@goodmis.org> -- Steve On Thu, 14 Nov 2019 13:37:19 +0000 Sudip Mukherjee <sudipm.mukherjee@gmail.com> wrote: > When we passed some location in DESTDIR, install_headers called > do_install with DESTDIR as part of the second argument. But do_install > is again using '$(DESTDIR_SQ)$2', so as a result the headers were > installed in a location $DESTDIR/$DESTDIR. In my testing I passed > DESTDIR=/home/sudip/test and the headers were installed in: > /home/sudip/test/home/sudip/test/usr/include/traceevent. > Lets remove DESTDIR from the second argument of do_install so that the > headers are installed in the correct location. > > Signed-off-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com> > --- > > Hi Steve, > sent this earlier as an attachment to an email thread, not sure if you > saw it, so sending it now properly. > The other problem with the pkgconfig, I guess we can live with that for > now as that folder is given by pc_path. > > tools/lib/traceevent/Makefile | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/tools/lib/traceevent/Makefile b/tools/lib/traceevent/Makefile > index 5315f3787f8d..cbb429f55062 100644 > --- a/tools/lib/traceevent/Makefile > +++ b/tools/lib/traceevent/Makefile > @@ -232,10 +232,10 @@ install_pkgconfig: > > install_headers: > $(call QUIET_INSTALL, headers) \ > - $(call do_install,event-parse.h,$(DESTDIR)$(includedir_SQ),644); \ > - $(call do_install,event-utils.h,$(DESTDIR)$(includedir_SQ),644); \ > - $(call do_install,trace-seq.h,$(DESTDIR)$(includedir_SQ),644); \ > - $(call do_install,kbuffer.h,$(DESTDIR)$(includedir_SQ),644) > + $(call do_install,event-parse.h,$(includedir_SQ),644); \ > + $(call do_install,event-utils.h,$(includedir_SQ),644); \ > + $(call do_install,trace-seq.h,$(includedir_SQ),644); \ > + $(call do_install,kbuffer.h,$(includedir_SQ),644) > > install: install_lib >
On November 14, 2019 7:40:22 PM GMT-03:00, Steven Rostedt <rostedt@goodmis.org> wrote: > >Arnaldo, > >Can you take this? Sure > >Reviewed-by: Steven Rostedt (VMware) <rostedt@goodmis.org> > >-- Steve > > >On Thu, 14 Nov 2019 13:37:19 +0000 >Sudip Mukherjee <sudipm.mukherjee@gmail.com> wrote: > >> When we passed some location in DESTDIR, install_headers called >> do_install with DESTDIR as part of the second argument. But >do_install >> is again using '$(DESTDIR_SQ)$2', so as a result the headers were >> installed in a location $DESTDIR/$DESTDIR. In my testing I passed >> DESTDIR=/home/sudip/test and the headers were installed in: >> /home/sudip/test/home/sudip/test/usr/include/traceevent. >> Lets remove DESTDIR from the second argument of do_install so that >the >> headers are installed in the correct location. >> >> Signed-off-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com> >> --- >> >> Hi Steve, >> sent this earlier as an attachment to an email thread, not sure if >you >> saw it, so sending it now properly. >> The other problem with the pkgconfig, I guess we can live with that >for >> now as that folder is given by pc_path. >> >> tools/lib/traceevent/Makefile | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/tools/lib/traceevent/Makefile >b/tools/lib/traceevent/Makefile >> index 5315f3787f8d..cbb429f55062 100644 >> --- a/tools/lib/traceevent/Makefile >> +++ b/tools/lib/traceevent/Makefile >> @@ -232,10 +232,10 @@ install_pkgconfig: >> >> install_headers: >> $(call QUIET_INSTALL, headers) \ >> - $(call do_install,event-parse.h,$(DESTDIR)$(includedir_SQ),644); \ >> - $(call do_install,event-utils.h,$(DESTDIR)$(includedir_SQ),644); \ >> - $(call do_install,trace-seq.h,$(DESTDIR)$(includedir_SQ),644); \ >> - $(call do_install,kbuffer.h,$(DESTDIR)$(includedir_SQ),644) >> + $(call do_install,event-parse.h,$(includedir_SQ),644); \ >> + $(call do_install,event-utils.h,$(includedir_SQ),644); \ >> + $(call do_install,trace-seq.h,$(includedir_SQ),644); \ >> + $(call do_install,kbuffer.h,$(includedir_SQ),644) >> >> install: install_lib >>
Em Thu, Nov 14, 2019 at 05:40:22PM -0500, Steven Rostedt escreveu: > > Arnaldo, > > Can you take this? > > Reviewed-by: Steven Rostedt (VMware) <rostedt@goodmis.org> Thanks, applied. - Arnaldo > -- Steve > > > On Thu, 14 Nov 2019 13:37:19 +0000 > Sudip Mukherjee <sudipm.mukherjee@gmail.com> wrote: > > > When we passed some location in DESTDIR, install_headers called > > do_install with DESTDIR as part of the second argument. But do_install > > is again using '$(DESTDIR_SQ)$2', so as a result the headers were > > installed in a location $DESTDIR/$DESTDIR. In my testing I passed > > DESTDIR=/home/sudip/test and the headers were installed in: > > /home/sudip/test/home/sudip/test/usr/include/traceevent. > > Lets remove DESTDIR from the second argument of do_install so that the > > headers are installed in the correct location. > > > > Signed-off-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com> > > --- > > > > Hi Steve, > > sent this earlier as an attachment to an email thread, not sure if you > > saw it, so sending it now properly. > > The other problem with the pkgconfig, I guess we can live with that for > > now as that folder is given by pc_path. > > > > tools/lib/traceevent/Makefile | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > diff --git a/tools/lib/traceevent/Makefile b/tools/lib/traceevent/Makefile > > index 5315f3787f8d..cbb429f55062 100644 > > --- a/tools/lib/traceevent/Makefile > > +++ b/tools/lib/traceevent/Makefile > > @@ -232,10 +232,10 @@ install_pkgconfig: > > > > install_headers: > > $(call QUIET_INSTALL, headers) \ > > - $(call do_install,event-parse.h,$(DESTDIR)$(includedir_SQ),644); \ > > - $(call do_install,event-utils.h,$(DESTDIR)$(includedir_SQ),644); \ > > - $(call do_install,trace-seq.h,$(DESTDIR)$(includedir_SQ),644); \ > > - $(call do_install,kbuffer.h,$(DESTDIR)$(includedir_SQ),644) > > + $(call do_install,event-parse.h,$(includedir_SQ),644); \ > > + $(call do_install,event-utils.h,$(includedir_SQ),644); \ > > + $(call do_install,trace-seq.h,$(includedir_SQ),644); \ > > + $(call do_install,kbuffer.h,$(includedir_SQ),644) > > > > install: install_lib > >
diff --git a/tools/lib/traceevent/Makefile b/tools/lib/traceevent/Makefile index 5315f3787f8d..cbb429f55062 100644 --- a/tools/lib/traceevent/Makefile +++ b/tools/lib/traceevent/Makefile @@ -232,10 +232,10 @@ install_pkgconfig: install_headers: $(call QUIET_INSTALL, headers) \ - $(call do_install,event-parse.h,$(DESTDIR)$(includedir_SQ),644); \ - $(call do_install,event-utils.h,$(DESTDIR)$(includedir_SQ),644); \ - $(call do_install,trace-seq.h,$(DESTDIR)$(includedir_SQ),644); \ - $(call do_install,kbuffer.h,$(DESTDIR)$(includedir_SQ),644) + $(call do_install,event-parse.h,$(includedir_SQ),644); \ + $(call do_install,event-utils.h,$(includedir_SQ),644); \ + $(call do_install,trace-seq.h,$(includedir_SQ),644); \ + $(call do_install,kbuffer.h,$(includedir_SQ),644) install: install_lib
When we passed some location in DESTDIR, install_headers called do_install with DESTDIR as part of the second argument. But do_install is again using '$(DESTDIR_SQ)$2', so as a result the headers were installed in a location $DESTDIR/$DESTDIR. In my testing I passed DESTDIR=/home/sudip/test and the headers were installed in: /home/sudip/test/home/sudip/test/usr/include/traceevent. Lets remove DESTDIR from the second argument of do_install so that the headers are installed in the correct location. Signed-off-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com> --- Hi Steve, sent this earlier as an attachment to an email thread, not sure if you saw it, so sending it now properly. The other problem with the pkgconfig, I guess we can live with that for now as that folder is given by pc_path. tools/lib/traceevent/Makefile | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)