All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/belr: fix BR2_SHARED_STATIC_LIBS build
@ 2022-04-10 17:24 Fabrice Fontaine
  2022-04-11  7:16 ` Peter Korsgaard
  2022-04-11  9:46 ` Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Fabrice Fontaine @ 2022-04-10 17:24 UTC (permalink / raw)
  To: buildroot; +Cc: Fabrice Fontaine

Fix the following build failure with BR2_SHARED_STATIC_LIBS:

CMake Error at src/CMakeLists.txt:56 (add_library):
  add_library cannot create target "belr" because another target with the
  same name already exists.  The existing target is a static library created
  in source directory

Fixes:
 - http://autobuild.buildroot.org/results/d1ef96c8f370b5a522985c37f1681dd10bbc15bb

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/belr/belr.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/belr/belr.mk b/package/belr/belr.mk
index f243830a85..1a849328ec 100644
--- a/package/belr/belr.mk
+++ b/package/belr/belr.mk
@@ -18,7 +18,7 @@ BELR_CONF_OPTS = \
 ifeq ($(BR2_STATIC_LIBS),y)
 BELR_CONF_OPTS += -DENABLE_SHARED=OFF -DENABLE_STATIC=ON
 else ifeq ($(BR2_SHARED_STATIC_LIBS),y)
-BELR_CONF_OPTS += -DENABLE_SHARED=ON -DENABLE_STATIC=ON
+BELR_CONF_OPTS += -DENABLE_SHARED=ON -DENABLE_STATIC=OFF
 else ifeq ($(BR2_SHARED_LIBS),y)
 BELR_CONF_OPTS += -DENABLE_SHARED=ON -DENABLE_STATIC=OFF
 endif
-- 
2.35.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/belr: fix BR2_SHARED_STATIC_LIBS build
  2022-04-10 17:24 [Buildroot] [PATCH 1/1] package/belr: fix BR2_SHARED_STATIC_LIBS build Fabrice Fontaine
@ 2022-04-11  7:16 ` Peter Korsgaard
  2022-04-11  9:46 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2022-04-11  7:16 UTC (permalink / raw)
  To: Fabrice Fontaine; +Cc: buildroot

>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > Fix the following build failure with BR2_SHARED_STATIC_LIBS:
 > CMake Error at src/CMakeLists.txt:56 (add_library):
 >   add_library cannot create target "belr" because another target with the
 >   same name already exists.  The existing target is a static library created
 >   in source directory

 > Fixes:
 >  - http://autobuild.buildroot.org/results/d1ef96c8f370b5a522985c37f1681dd10bbc15bb

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
 > ---
 >  package/belr/belr.mk | 2 +-
 >  1 file changed, 1 insertion(+), 1 deletion(-)

 > diff --git a/package/belr/belr.mk b/package/belr/belr.mk
 > index f243830a85..1a849328ec 100644
 > --- a/package/belr/belr.mk
 > +++ b/package/belr/belr.mk
 > @@ -18,7 +18,7 @@ BELR_CONF_OPTS = \
 >  ifeq ($(BR2_STATIC_LIBS),y)
 >  BELR_CONF_OPTS += -DENABLE_SHARED=OFF -DENABLE_STATIC=ON
 >  else ifeq ($(BR2_SHARED_STATIC_LIBS),y)
 > -BELR_CONF_OPTS += -DENABLE_SHARED=ON -DENABLE_STATIC=ON
 > +BELR_CONF_OPTS += -DENABLE_SHARED=ON -DENABLE_STATIC=OFF
 >  else ifeq ($(BR2_SHARED_LIBS),y)
 >  BELR_CONF_OPTS += -DENABLE_SHARED=ON -DENABLE_STATIC=OFF
 >  endif

This makes the BR2_SHARED_STATIC_LIBS and BR2_SHARED_LIBS conditionals
the same, so I just changed it to:

ifeq ($(BR2_STATIC_LIBS),y)
BELR_CONF_OPTS += -DENABLE_SHARED=OFF -DENABLE_STATIC=ON
else
# cannot build static and shared together
BELR_CONF_OPTS += -DENABLE_SHARED=ON -DENABLE_STATIC=OFF
endif

And committed, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/belr: fix BR2_SHARED_STATIC_LIBS build
  2022-04-10 17:24 [Buildroot] [PATCH 1/1] package/belr: fix BR2_SHARED_STATIC_LIBS build Fabrice Fontaine
  2022-04-11  7:16 ` Peter Korsgaard
@ 2022-04-11  9:46 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2022-04-11  9:46 UTC (permalink / raw)
  To: Fabrice Fontaine; +Cc: buildroot

>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > Fix the following build failure with BR2_SHARED_STATIC_LIBS:
 > CMake Error at src/CMakeLists.txt:56 (add_library):
 >   add_library cannot create target "belr" because another target with the
 >   same name already exists.  The existing target is a static library created
 >   in source directory

 > Fixes:
 >  - http://autobuild.buildroot.org/results/d1ef96c8f370b5a522985c37f1681dd10bbc15bb

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed to 2022.02.x, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-04-11  9:46 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-10 17:24 [Buildroot] [PATCH 1/1] package/belr: fix BR2_SHARED_STATIC_LIBS build Fabrice Fontaine
2022-04-11  7:16 ` Peter Korsgaard
2022-04-11  9:46 ` Peter Korsgaard

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.