* [Buildroot] [PATCH] package/civetweb: fix link failure due to missing OpenSSL dependency
@ 2019-04-09 21:10 Giulio Benetti
2019-04-09 21:28 ` Thomas Petazzoni
2019-04-14 21:26 ` Peter Korsgaard
0 siblings, 2 replies; 4+ messages in thread
From: Giulio Benetti @ 2019-04-09 21:10 UTC (permalink / raw)
To: buildroot
During linking one OpenSSL dependecy is missing(-latomic) on linking
library list.
- Substitute explicit library list with `pkg-config libssl` when
BR2_PACKAGE_OPENSSL is enabled. In such way all needed libraries will be
included in linking list.
- Add also `host-pkgconf` to CIVETWEB_DEPENDENCIES if
BR2_PACKAGE_OPENSSL is enabled to make it available for previous point.
Fixes:
http://autobuild.buildroot.net/results/b2e/b2e210bdefe84f4ec9cfda79a33d81788fb7e66c/
Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
---
package/civetweb/civetweb.mk | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/package/civetweb/civetweb.mk b/package/civetweb/civetweb.mk
index fe1cdd5675..c999ec89be 100644
--- a/package/civetweb/civetweb.mk
+++ b/package/civetweb/civetweb.mk
@@ -32,8 +32,8 @@ endif
ifeq ($(BR2_PACKAGE_OPENSSL),y)
CIVETWEB_COPT += -DNO_SSL_DL
-CIVETWEB_LIBS += -lssl -lcrypto -lz
-CIVETWEB_DEPENDENCIES += openssl
+CIVETWEB_LIBS += `$(HOST_DIR)/usr/bin/pkg-config --libs libssl`
+CIVETWEB_DEPENDENCIES += openssl host-pkgconf
else
CIVETWEB_COPT += -DNO_SSL
endif
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] package/civetweb: fix link failure due to missing OpenSSL dependency
2019-04-09 21:10 [Buildroot] [PATCH] package/civetweb: fix link failure due to missing OpenSSL dependency Giulio Benetti
@ 2019-04-09 21:28 ` Thomas Petazzoni
2019-04-10 10:06 ` Giulio Benetti
2019-04-14 21:26 ` Peter Korsgaard
1 sibling, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2019-04-09 21:28 UTC (permalink / raw)
To: buildroot
On Tue, 9 Apr 2019 23:10:32 +0200
Giulio Benetti <giulio.benetti@micronovasrl.com> wrote:
> During linking one OpenSSL dependecy is missing(-latomic) on linking
> library list.
>
> - Substitute explicit library list with `pkg-config libssl` when
> BR2_PACKAGE_OPENSSL is enabled. In such way all needed libraries will be
> included in linking list.
> - Add also `host-pkgconf` to CIVETWEB_DEPENDENCIES if
> BR2_PACKAGE_OPENSSL is enabled to make it available for previous point.
>
> Fixes:
> http://autobuild.buildroot.net/results/b2e/b2e210bdefe84f4ec9cfda79a33d81788fb7e66c/
>
> Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
Applied to master, with one small change, see below.
> +CIVETWEB_LIBS += `$(HOST_DIR)/usr/bin/pkg-config --libs libssl`
$(HOST_DIR)/usr no longer really exists, it's a symlink to $(HOST_DIR),
so this path should have been just $(HOST_DIR)/bin/pkg-config.
However, pkgconf.mk precisely provides the PKG_CONFIG_HOST_BINARY
variable, which contains exactly the path to pkg-config. So I've used
that instead.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] package/civetweb: fix link failure due to missing OpenSSL dependency
2019-04-09 21:28 ` Thomas Petazzoni
@ 2019-04-10 10:06 ` Giulio Benetti
0 siblings, 0 replies; 4+ messages in thread
From: Giulio Benetti @ 2019-04-10 10:06 UTC (permalink / raw)
To: buildroot
Il 09/04/2019 23:28, Thomas Petazzoni ha scritto:> $(HOST_DIR)/usr no
longer really exists, it's a symlink to $(HOST_DIR),
> so this path should have been just $(HOST_DIR)/bin/pkg-config.
>
> However, pkgconf.mk precisely provides the PKG_CONFIG_HOST_BINARY
> variable, which contains exactly the path to pkg-config. So I've used
> that instead.
Ah I've missed that variable good to know.
Thank you
--
Giulio Benetti
CTO
MICRONOVA SRL
Sede: Via A. Niedda 3 - 35010 Vigonza (PD)
Tel. 049/8931563 - Fax 049/8931346
Cod.Fiscale - P.IVA 02663420285
Capitale Sociale ? 26.000 i.v.
Iscritta al Reg. Imprese di Padova N. 02663420285
Numero R.E.A. 258642
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] package/civetweb: fix link failure due to missing OpenSSL dependency
2019-04-09 21:10 [Buildroot] [PATCH] package/civetweb: fix link failure due to missing OpenSSL dependency Giulio Benetti
2019-04-09 21:28 ` Thomas Petazzoni
@ 2019-04-14 21:26 ` Peter Korsgaard
1 sibling, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2019-04-14 21:26 UTC (permalink / raw)
To: buildroot
>>>>> "Giulio" == Giulio Benetti <giulio.benetti@micronovasrl.com> writes:
> During linking one OpenSSL dependecy is missing(-latomic) on linking
> library list.
> - Substitute explicit library list with `pkg-config libssl` when
> BR2_PACKAGE_OPENSSL is enabled. In such way all needed libraries will be
> included in linking list.
> - Add also `host-pkgconf` to CIVETWEB_DEPENDENCIES if
> BR2_PACKAGE_OPENSSL is enabled to make it available for previous point.
> Fixes:
> http://autobuild.buildroot.net/results/b2e/b2e210bdefe84f4ec9cfda79a33d81788fb7e66c/
> Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
Committed to 2019.02.x, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-04-14 21:26 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-09 21:10 [Buildroot] [PATCH] package/civetweb: fix link failure due to missing OpenSSL dependency Giulio Benetti
2019-04-09 21:28 ` Thomas Petazzoni
2019-04-10 10:06 ` Giulio Benetti
2019-04-14 21:26 ` 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.