Message ID | 20180508161136.13531-1-fontaine.fabrice@gmail.com |
---|---|
State | Superseded |
Headers | show |
Series | [1/1] libcap: only install shared version | expand |
Hello, On Tue, 8 May 2018 18:11:36 +0200, Fabrice Fontaine wrote: > diff --git a/package/libcap/libcap.mk b/package/libcap/libcap.mk > index 48c2cb3456..247fd8a58f 100644 > --- a/package/libcap/libcap.mk > +++ b/package/libcap/libcap.mk > @@ -20,8 +20,12 @@ LIBCAP_MAKE_TARGET = libcap.a libcap.pc > LIBCAP_MAKE_INSTALL_TARGET = install-static > else > LIBCAP_MAKE_TARGET = all > +ifeq ($(BR2_SHARED_LIBS),y) > +LIBCAP_MAKE_INSTALL_TARGET = install-shared > +else > LIBCAP_MAKE_INSTALL_TARGET = install > endif > +endif I think we could use a clearer approach: ifeq ($(BR2_STATIC_LIBS),y) LIBCAP_MAKE_TARGET = libcap.a libcap.pc LIBCAP_MAKE_INSTALL_TARGET = install-static else if ($(BR2_SHARED_LIBS),y) LIBCAP_MAKE_TARGET = all LIBCAP_MAKE_INSTALL_TARGET = install-shared else LIBCAP_MAKE_TARGET = all LIBCAP_MAKE_INSTALL_TARGET = install endif is a lot clearer to me. Best regards, Thomas
diff --git a/package/libcap/libcap.mk b/package/libcap/libcap.mk index 48c2cb3456..247fd8a58f 100644 --- a/package/libcap/libcap.mk +++ b/package/libcap/libcap.mk @@ -20,8 +20,12 @@ LIBCAP_MAKE_TARGET = libcap.a libcap.pc LIBCAP_MAKE_INSTALL_TARGET = install-static else LIBCAP_MAKE_TARGET = all +ifeq ($(BR2_SHARED_LIBS),y) +LIBCAP_MAKE_INSTALL_TARGET = install-shared +else LIBCAP_MAKE_INSTALL_TARGET = install endif +endif LIBCAP_MAKE_FLAGS = \ BUILD_CC="$(HOSTCC)" \
If BR2_SHARED_LIBS is set, only install shared version of library (continue to build both libraries through all target as there is no libcap.so target but only a libcap.so.$(VERSION).$(MINOR)) Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> --- package/libcap/libcap.mk | 4 ++++ 1 file changed, 4 insertions(+)