diff mbox series

[v1] package/ser2net: fix gensio detection with openssl enabled

Message ID 20200308165149.31293-1-ps.report@gmx.net
State Accepted
Headers show
Series [v1] package/ser2net: fix gensio detection with openssl enabled | expand

Commit Message

Peter Seiderer March 8, 2020, 4:51 p.m. UTC
Add openssl linker flags via LIBS to fix configure gensio
library detection.

Fixes:

  http://autobuild.buildroot.net/results/66e0d3e0a2a8dc5a62c267d16a53216f0f2ce8dd

  checking gensio/gensio.h usability... yes
  checking gensio/gensio.h presence... yes
  checking for gensio/gensio.h... yes
  checking for str_to_gensio in -lgensio... no
  configure: error: libgensio won't link, please install gensio dev package

The build/ser2net-4.1.1/config.log files states:

  .../arm-buildroot-linux-uclibcgnueabi/bin/ld: .../host/arm-buildroot-linux-uclibcgnueabi/sysroot/usr/lib/libgensio.a(gensio_filter_ssl.o): in function `gensio_do_ssl_init':
  gensio_filter_ssl.c:(.text+0x34): undefined reference to `OPENSSL_init_ssl'

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
 package/ser2net/ser2net.mk | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Peter Korsgaard March 8, 2020, 8:38 p.m. UTC | #1
>>>>> "Peter" == Peter Seiderer <ps.report@gmx.net> writes:

 > Add openssl linker flags via LIBS to fix configure gensio
 > library detection.

 > Fixes:

 >   http://autobuild.buildroot.net/results/66e0d3e0a2a8dc5a62c267d16a53216f0f2ce8dd

 >   checking gensio/gensio.h usability... yes
 >   checking gensio/gensio.h presence... yes
 >   checking for gensio/gensio.h... yes
 >   checking for str_to_gensio in -lgensio... no
 >   configure: error: libgensio won't link, please install gensio dev package

 > The build/ser2net-4.1.1/config.log files states:

 >   .../arm-buildroot-linux-uclibcgnueabi/bin/ld: .../host/arm-buildroot-linux-uclibcgnueabi/sysroot/usr/lib/libgensio.a(gensio_filter_ssl.o): in function `gensio_do_ssl_init':
 >   gensio_filter_ssl.c:(.text+0x34): undefined reference to `OPENSSL_init_ssl'

 > Signed-off-by: Peter Seiderer <ps.report@gmx.net>

The real solution would be for gensio to provide a .pc file / ser2net
use pkg-config, but OK - This is good enough for 2020.02.
diff mbox series

Patch

diff --git a/package/ser2net/ser2net.mk b/package/ser2net/ser2net.mk
index 2ea68ccc43..94035b2184 100644
--- a/package/ser2net/ser2net.mk
+++ b/package/ser2net/ser2net.mk
@@ -16,6 +16,11 @@  else
 SER2NET_CONF_OPTS += --without-pthreads
 endif
 
+# fix gensio detection with openssl enabled
+ifeq ($(BR2_PACKAGE_OPENSSL),y)
+SER2NET_CONF_ENV += LIBS="`$(PKG_CONFIG_HOST_BINARY) --libs openssl`"
+endif
+
 define SER2NET_INSTALL_INIT_SYSV
 	$(INSTALL) -D -m 755 package/ser2net/S50ser2net \
 		$(TARGET_DIR)/etc/init.d/S50ser2net