* [Buildroot] [PATCH 1/1] jimtcl: change the way of define JIMTCL_INSTALL_LIB variable @ 2015-01-15 17:18 Jerzy Grzegorek 2015-01-18 15:54 ` Yann E. MORIN 2015-04-01 21:18 ` Thomas Petazzoni 0 siblings, 2 replies; 4+ messages in thread From: Jerzy Grzegorek @ 2015-01-15 17:18 UTC (permalink / raw) To: buildroot Signed-off-by: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net> --- package/jimtcl/jimtcl.mk | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/package/jimtcl/jimtcl.mk b/package/jimtcl/jimtcl.mk index 75c0382..c2e04ff 100644 --- a/package/jimtcl/jimtcl.mk +++ b/package/jimtcl/jimtcl.mk @@ -32,10 +32,11 @@ JIMTCL_INSTALL_LIB = else JIMTCL_SHARED = --shared JIMTCL_LIB = so.$(JIMTCL_VERSION) -JIMTCL_INSTALL_LIB = \ +define JIMTCL_INSTALL_LIB $(INSTALL) -D $(@D)/libjim.$(JIMTCL_LIB) \ - $(TARGET_DIR)/usr/lib/libjim.$(JIMTCL_LIB); \ + $(TARGET_DIR)/usr/lib/libjim.$(JIMTCL_LIB) ln -s libjim.$(JIMTCL_LIB) $(TARGET_DIR)/usr/lib/libjim.so +endef endif define JIMTCL_CONFIGURE_CMDS -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] jimtcl: change the way of define JIMTCL_INSTALL_LIB variable 2015-01-15 17:18 [Buildroot] [PATCH 1/1] jimtcl: change the way of define JIMTCL_INSTALL_LIB variable Jerzy Grzegorek @ 2015-01-18 15:54 ` Yann E. MORIN 2015-01-19 7:23 ` Jerzy Grzegorek 2015-04-01 21:18 ` Thomas Petazzoni 1 sibling, 1 reply; 4+ messages in thread From: Yann E. MORIN @ 2015-01-18 15:54 UTC (permalink / raw) To: buildroot Jerzy, All, On 2015-01-15 18:18 +0100, Jerzy Grzegorek spake thusly: > Signed-off-by: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net> Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr> However, there is something really fishy in _INSTALL_STAGING_CMDS: define JIMTCL_INSTALL_STAGING_CMDS for i in $(JIMTCL_HEADERS_TO_INSTALL); do \ cp -a $(@D)/$$i $(STAGING_DIR)/usr/include/ || exit 1 ; \ done; \ $(INSTALL) -D $(@D)/libjim.$(JIMTCL_LIB) $(STAGING_DIR)/usr/lib/libjim.$(JIMTCL_LIB); ln -s libjim.$(JIMTCL_LIB) $(STAGING_DIR)/usr/lib/libjim.so endef In case we're static-only, JIMTCL_LIB=a , so we're linking the static library to the shared one. Surely this is just busted. no? While you're at touching jimctl, would you care having a look, please? Thanks! :-) Regards, Yann E. MORIN. > --- > package/jimtcl/jimtcl.mk | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/package/jimtcl/jimtcl.mk b/package/jimtcl/jimtcl.mk > index 75c0382..c2e04ff 100644 > --- a/package/jimtcl/jimtcl.mk > +++ b/package/jimtcl/jimtcl.mk > @@ -32,10 +32,11 @@ JIMTCL_INSTALL_LIB = > else > JIMTCL_SHARED = --shared > JIMTCL_LIB = so.$(JIMTCL_VERSION) > -JIMTCL_INSTALL_LIB = \ > +define JIMTCL_INSTALL_LIB > $(INSTALL) -D $(@D)/libjim.$(JIMTCL_LIB) \ > - $(TARGET_DIR)/usr/lib/libjim.$(JIMTCL_LIB); \ > + $(TARGET_DIR)/usr/lib/libjim.$(JIMTCL_LIB) > ln -s libjim.$(JIMTCL_LIB) $(TARGET_DIR)/usr/lib/libjim.so > +endef > endif > > define JIMTCL_CONFIGURE_CMDS > -- > 1.7.9.5 > > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------' ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] jimtcl: change the way of define JIMTCL_INSTALL_LIB variable 2015-01-18 15:54 ` Yann E. MORIN @ 2015-01-19 7:23 ` Jerzy Grzegorek 0 siblings, 0 replies; 4+ messages in thread From: Jerzy Grzegorek @ 2015-01-19 7:23 UTC (permalink / raw) To: buildroot Hi Yann, > Jerzy, All, > > On 2015-01-15 18:18 +0100, Jerzy Grzegorek spake thusly: >> Signed-off-by: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net> > Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr> > > However, there is something really fishy in _INSTALL_STAGING_CMDS: > > define JIMTCL_INSTALL_STAGING_CMDS > for i in $(JIMTCL_HEADERS_TO_INSTALL); do \ > cp -a $(@D)/$$i $(STAGING_DIR)/usr/include/ || exit 1 ; \ > done; \ > $(INSTALL) -D $(@D)/libjim.$(JIMTCL_LIB) $(STAGING_DIR)/usr/lib/libjim.$(JIMTCL_LIB); > ln -s libjim.$(JIMTCL_LIB) $(STAGING_DIR)/usr/lib/libjim.so > endef > > In case we're static-only, JIMTCL_LIB=a , so we're linking the static > library to the shared one. Surely this is just busted. no? Yes, indeed. > > While you're at touching jimctl, would you care having a look, please? OK, will fix. Thanks for review. Regards, Jerzy > > Thanks! :-) > > Regards, > Yann E. MORIN. > >> --- >> package/jimtcl/jimtcl.mk | 5 +++-- >> 1 file changed, 3 insertions(+), 2 deletions(-) >> >> diff --git a/package/jimtcl/jimtcl.mk b/package/jimtcl/jimtcl.mk >> index 75c0382..c2e04ff 100644 >> --- a/package/jimtcl/jimtcl.mk >> +++ b/package/jimtcl/jimtcl.mk >> @@ -32,10 +32,11 @@ JIMTCL_INSTALL_LIB = >> else >> JIMTCL_SHARED = --shared >> JIMTCL_LIB = so.$(JIMTCL_VERSION) >> -JIMTCL_INSTALL_LIB = \ >> +define JIMTCL_INSTALL_LIB >> $(INSTALL) -D $(@D)/libjim.$(JIMTCL_LIB) \ >> - $(TARGET_DIR)/usr/lib/libjim.$(JIMTCL_LIB); \ >> + $(TARGET_DIR)/usr/lib/libjim.$(JIMTCL_LIB) >> ln -s libjim.$(JIMTCL_LIB) $(TARGET_DIR)/usr/lib/libjim.so >> +endef >> endif >> >> define JIMTCL_CONFIGURE_CMDS >> -- >> 1.7.9.5 >> >> _______________________________________________ >> buildroot mailing list >> buildroot at busybox.net >> http://lists.busybox.net/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] jimtcl: change the way of define JIMTCL_INSTALL_LIB variable 2015-01-15 17:18 [Buildroot] [PATCH 1/1] jimtcl: change the way of define JIMTCL_INSTALL_LIB variable Jerzy Grzegorek 2015-01-18 15:54 ` Yann E. MORIN @ 2015-04-01 21:18 ` Thomas Petazzoni 1 sibling, 0 replies; 4+ messages in thread From: Thomas Petazzoni @ 2015-04-01 21:18 UTC (permalink / raw) To: buildroot Dear Jerzy Grzegorek, On Thu, 15 Jan 2015 18:18:14 +0100, Jerzy Grzegorek wrote: > Signed-off-by: Jerzy Grzegorek <jerzy.grzegorek@trzebnica.net> > --- > package/jimtcl/jimtcl.mk | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) Applied, thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-04-01 21:18 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-01-15 17:18 [Buildroot] [PATCH 1/1] jimtcl: change the way of define JIMTCL_INSTALL_LIB variable Jerzy Grzegorek 2015-01-18 15:54 ` Yann E. MORIN 2015-01-19 7:23 ` Jerzy Grzegorek 2015-04-01 21:18 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox