Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/sofia-sip: needs OpenSSL
@ 2026-09-17  0:01 Bernd Kuhls
  2026-09-17  6:37 ` Thomas Petazzoni via buildroot
  2026-09-25 14:46 ` Raphaël Mélotte via buildroot
  0 siblings, 2 replies; 3+ messages in thread
From: Bernd Kuhls @ 2026-09-17  0:01 UTC (permalink / raw)
  To: buildroot; +Cc: Gregory Dymarek

Buildroot commit bb254e2304274dc308e3d8699e92ea77e4189fb7 bumped the
package to version 1.13.18 which includes upstream commit
https://github.com/freeswitch/sofia-sip/commit/8081a1a6d05f174cb902664f73182c32fd2eee0f
that added the unconditional usage of OpenSSL.

Tested with both LibreSSL and OpenSSL, the latter with all suboptions
disabled.

Fixes:
https://autobuild.buildroot.net/results/afe/afe7b327f12db86e4dc31ecc25b576bff5c4a0bb/

Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
---
 package/sofia-sip/Config.in    |  1 +
 package/sofia-sip/sofia-sip.mk | 18 +++++-------------
 2 files changed, 6 insertions(+), 13 deletions(-)

diff --git a/package/sofia-sip/Config.in b/package/sofia-sip/Config.in
index 2ad1940833..37961273f3 100644
--- a/package/sofia-sip/Config.in
+++ b/package/sofia-sip/Config.in
@@ -1,6 +1,7 @@
 config BR2_PACKAGE_SOFIA_SIP
 	bool "sofia-sip"
 	depends on BR2_TOOLCHAIN_HAS_THREADS
+	select BR2_PACKAGE_OPENSSL
 	help
 	  Sofia-SIP is an open-source SIP User-Agent library,
 	  compliant with the IETF RFC3261 specification (see the
diff --git a/package/sofia-sip/sofia-sip.mk b/package/sofia-sip/sofia-sip.mk
index 1d47b43fe8..b8679f6961 100644
--- a/package/sofia-sip/sofia-sip.mk
+++ b/package/sofia-sip/sofia-sip.mk
@@ -9,11 +9,14 @@ SOFIA_SIP_SITE = $(call github,freeswitch,sofia-sip,v$(SOFIA_SIP_VERSION))
 SOFIA_SIP_INSTALL_STAGING = YES
 # Fetched from github, no pre-generated configure script provided
 SOFIA_SIP_AUTORECONF = YES
-SOFIA_SIP_DEPENDENCIES = host-pkgconf
+SOFIA_SIP_DEPENDENCIES = host-pkgconf openssl
 SOFIA_SIP_LICENSE = LGPL-2.1+
 SOFIA_SIP_LICENSE_FILES = COPYING COPYRIGHTS
 SOFIA_SIP_CPE_ID_VENDOR = signalwire
-SOFIA_SIP_CONF_OPTS = --with-doxygen=no
+SOFIA_SIP_CONF_OPTS = \
+	--with-doxygen=no \
+	--enable-nth \
+	--with-openssl=pkg-config
 
 ifeq ($(BR2_PACKAGE_LIBGLIB2),y)
 SOFIA_SIP_CONF_OPTS += --with-glib
@@ -22,15 +25,4 @@ else
 SOFIA_SIP_CONF_OPTS += --without-glib
 endif
 
-ifeq ($(BR2_PACKAGE_OPENSSL),y)
-SOFIA_SIP_CONF_OPTS += \
-	--enable-nth \
-	--with-openssl=pkg-config
-SOFIA_SIP_DEPENDENCIES += openssl
-else
-SOFIA_SIP_CONF_OPTS += \
-	--disable-nth \
-	--without-openssl
-endif
-
 $(eval $(autotools-package))
-- 
2.47.3

_______________________________________________
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/sofia-sip: needs OpenSSL
  2026-09-17  0:01 [Buildroot] [PATCH 1/1] package/sofia-sip: needs OpenSSL Bernd Kuhls
@ 2026-09-17  6:37 ` Thomas Petazzoni via buildroot
  2026-09-25 14:46 ` Raphaël Mélotte via buildroot
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni via buildroot @ 2026-09-17  6:37 UTC (permalink / raw)
  To: Bernd Kuhls; +Cc: buildroot, Gregory Dymarek

On Thu, Sep 17, 2026 at 02:01:52AM +0200, Bernd Kuhls wrote:
> Buildroot commit bb254e2304274dc308e3d8699e92ea77e4189fb7 bumped the
> package to version 1.13.18 which includes upstream commit
> https://github.com/freeswitch/sofia-sip/commit/8081a1a6d05f174cb902664f73182c32fd2eee0f
> that added the unconditional usage of OpenSSL.
> 
> Tested with both LibreSSL and OpenSSL, the latter with all suboptions
> disabled.
> 
> Fixes:
> https://autobuild.buildroot.net/results/afe/afe7b327f12db86e4dc31ecc25b576bff5c4a0bb/
> 
> Signed-off-by: Bernd Kuhls <bernd@kuhls.net>

Applied to master, thanks! However, could you submit a bug report
upstream? Indeed, it is not normal for the build to blow up: the lack
of OpenSSL should be detected at the configure.ac level. This bug
report will also allow to check whether making OpenSSL a mandatory
dependency is really a conscious decision from the upstream
maintainers.

Thanks a lot!

Thomas Petazzoni
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
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/sofia-sip: needs OpenSSL
  2026-09-17  0:01 [Buildroot] [PATCH 1/1] package/sofia-sip: needs OpenSSL Bernd Kuhls
  2026-09-17  6:37 ` Thomas Petazzoni via buildroot
@ 2026-09-25 14:46 ` Raphaël Mélotte via buildroot
  1 sibling, 0 replies; 3+ messages in thread
From: Raphaël Mélotte via buildroot @ 2026-09-25 14:46 UTC (permalink / raw)
  To: buildroot, Bernd Kuhls; +Cc: Raphaël Mélotte

In reply of:
> Buildroot commit bb254e2304274dc308e3d8699e92ea77e4189fb7 bumped the
> package to version 1.13.18 which includes upstream commit
> https://github.com/freeswitch/sofia-sip/commit/8081a1a6d05f174cb902664f73182c32fd2eee0f
> that added the unconditional usage of OpenSSL.
> 
> Tested with both LibreSSL and OpenSSL, the latter with all suboptions
> disabled.
> 
> Fixes:
> https://autobuild.buildroot.net/results/afe/afe7b327f12db86e4dc31ecc25b576bff5c4a0bb/
> 
> Signed-off-by: Bernd Kuhls <bernd@kuhls.net>

Applied to 2026.08.x. Thanks

> ---
>  package/sofia-sip/Config.in    |  1 +
>  package/sofia-sip/sofia-sip.mk | 18 +++++-------------
>  2 files changed, 6 insertions(+), 13 deletions(-)
> 
> diff --git a/package/sofia-sip/Config.in b/package/sofia-sip/Config.in
> index 2ad1940833..37961273f3 100644
> --- a/package/sofia-sip/Config.in
> +++ b/package/sofia-sip/Config.in
> @@ -1,6 +1,7 @@
>  config BR2_PACKAGE_SOFIA_SIP
>  	bool "sofia-sip"
>  	depends on BR2_TOOLCHAIN_HAS_THREADS
> +	select BR2_PACKAGE_OPENSSL
>  	help
>  	  Sofia-SIP is an open-source SIP User-Agent library,
>  	  compliant with the IETF RFC3261 specification (see the
> diff --git a/package/sofia-sip/sofia-sip.mk b/package/sofia-sip/sofia-sip.mk
> index 1d47b43fe8..b8679f6961 100644
> --- a/package/sofia-sip/sofia-sip.mk
> +++ b/package/sofia-sip/sofia-sip.mk
> @@ -9,11 +9,14 @@ SOFIA_SIP_SITE = $(call github,freeswitch,sofia-sip,v$(SOFIA_SIP_VERSION))
>  SOFIA_SIP_INSTALL_STAGING = YES
>  # Fetched from github, no pre-generated configure script provided
>  SOFIA_SIP_AUTORECONF = YES
> -SOFIA_SIP_DEPENDENCIES = host-pkgconf
> +SOFIA_SIP_DEPENDENCIES = host-pkgconf openssl
>  SOFIA_SIP_LICENSE = LGPL-2.1+
>  SOFIA_SIP_LICENSE_FILES = COPYING COPYRIGHTS
>  SOFIA_SIP_CPE_ID_VENDOR = signalwire
> -SOFIA_SIP_CONF_OPTS = --with-doxygen=no
> +SOFIA_SIP_CONF_OPTS = \
> +	--with-doxygen=no \
> +	--enable-nth \
> +	--with-openssl=pkg-config
>  
>  ifeq ($(BR2_PACKAGE_LIBGLIB2),y)
>  SOFIA_SIP_CONF_OPTS += --with-glib
> @@ -22,15 +25,4 @@ else
>  SOFIA_SIP_CONF_OPTS += --without-glib
>  endif
>  
> -ifeq ($(BR2_PACKAGE_OPENSSL),y)
> -SOFIA_SIP_CONF_OPTS += \
> -	--enable-nth \
> -	--with-openssl=pkg-config
> -SOFIA_SIP_DEPENDENCIES += openssl
> -else
> -SOFIA_SIP_CONF_OPTS += \
> -	--disable-nth \
> -	--without-openssl
> -endif
> -
>  $(eval $(autotools-package))
_______________________________________________
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:[~2026-09-25 14:46 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-17  0:01 [Buildroot] [PATCH 1/1] package/sofia-sip: needs OpenSSL Bernd Kuhls
2026-09-17  6:37 ` Thomas Petazzoni via buildroot
2026-09-25 14:46 ` Raphaël Mélotte via buildroot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox