* [PATCH] scripts: use pkg-config to locate libcrypto
@ 2018-11-22 15:45 Rolf Eike Beer
2018-11-22 16:16 ` David Woodhouse
0 siblings, 1 reply; 8+ messages in thread
From: Rolf Eike Beer @ 2018-11-22 15:45 UTC (permalink / raw)
To: Linux Kernel Developers List; +Cc: David Howells, David Woodhouse, keyrings
Otherwise build fails if the headers are not in the default location. While at
it also ask pkg-config for the libs, with fallback to the existing value.
Signed-off-by: Rolf Eike Beer <eb@emlix.com>
Cc: stable@vger.kernel.org
---
scripts/Makefile | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/scripts/Makefile b/scripts/Makefile
index ece52ff20171..a303f478a65b 100644
--- a/scripts/Makefile
+++ b/scripts/Makefile
@@ -8,6 +8,8 @@
# conmakehash: Create chartable
# conmakehash: Create arrays for initializing the kernel console tables
+PKG_CONFIG?= pkg-config
+
HOST_EXTRACFLAGS += -I$(srctree)/tools/include
hostprogs-$(CONFIG_BUILD_BIN2C) += bin2c
@@ -23,8 +25,9 @@ hostprogs-$(CONFIG_SYSTEM_EXTRA_CERTIFICATE) += insert-sys-cert
HOSTCFLAGS_sortextable.o = -I$(srctree)/tools/include
HOSTCFLAGS_asn1_compiler.o = -I$(srctree)/include
-HOSTLDLIBS_sign-file = -lcrypto
-HOSTLDLIBS_extract-cert = -lcrypto
+HOSTLDLIBS_sign-file = $(shell $(PKG_CONFIG) --libs libcrypto 2> /dev/null || -lcrypto)
+HOSTCFLAGS_extract-cert.o = $(shell $(PKG_CONFIG) --cflags libcrypto 2> /dev/null)
+HOSTLDLIBS_extract-cert = $(shell $(PKG_CONFIG) --libs libcrypto 2> /dev/null || -lcrypto)
always := $(hostprogs-y) $(hostprogs-m)
--
2.19.1
--
Rolf Eike Beer, emlix GmbH, http://www.emlix.com
Fon +49 551 30664-0, Fax +49 551 30664-11
Gothaer Platz 3, 37083 Göttingen, Germany
Sitz der Gesellschaft: Göttingen, Amtsgericht Göttingen HR B 3160
Geschäftsführung: Heike Jordan, Dr. Uwe Kracke – Ust-IdNr.: DE 205 198 055
emlix - smart embedded open source
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] scripts: use pkg-config to locate libcrypto
2018-11-22 15:45 [PATCH] scripts: use pkg-config to locate libcrypto Rolf Eike Beer
@ 2018-11-22 16:16 ` David Woodhouse
2018-11-22 16:42 ` [PATCH v2] " Rolf Eike Beer
0 siblings, 1 reply; 8+ messages in thread
From: David Woodhouse @ 2018-11-22 16:16 UTC (permalink / raw)
To: Rolf Eike Beer, Linux Kernel Developers List; +Cc: David Howells, keyrings
[-- Attachment #1: Type: text/plain, Size: 574 bytes --]
On Thu, 2018-11-22 at 16:45 +0100, Rolf Eike Beer wrote:
> -HOSTLDLIBS_sign-file = -lcrypto
> -HOSTLDLIBS_extract-cert = -lcrypto
> +HOSTLDLIBS_sign-file = $(shell $(PKG_CONFIG) --libs libcrypto 2> /dev/null || -lcrypto)
> +HOSTCFLAGS_extract-cert.o = $(shell $(PKG_CONFIG) --cflags libcrypto 2> /dev/null)
> +HOSTLDLIBS_extract-cert = $(shell $(PKG_CONFIG) --libs libcrypto 2> /dev/null || -lcrypto)
Looks good; thanks. But could you put it into intermediate
CRYPTO_CFLAGS and CRYPTO_LIBS variables instead of calculating the
latter in two separate callouts?
[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 5213 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2] scripts: use pkg-config to locate libcrypto
2018-11-22 16:16 ` David Woodhouse
@ 2018-11-22 16:42 ` Rolf Eike Beer
[not found] ` <20181124055339.E3B9120881@mail.kernel.org>
2019-06-06 7:55 ` [PATCH v2 RESEND] " Rolf Eike Beer
0 siblings, 2 replies; 8+ messages in thread
From: Rolf Eike Beer @ 2018-11-22 16:42 UTC (permalink / raw)
To: David Woodhouse; +Cc: Linux Kernel Developers List, David Howells, keyrings
Otherwise build fails if the headers are not in the default location. While at
it also ask pkg-config for the libs, with fallback to the existing value.
Signed-off-by: Rolf Eike Beer <eb@emlix.com>
Cc: stable@vger.kernel.org
---
scripts/Makefile | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/scripts/Makefile b/scripts/Makefile
index ece52ff20171..769fa6b7b685 100644
--- a/scripts/Makefile
+++ b/scripts/Makefile
@@ -8,7 +8,11 @@
# conmakehash: Create chartable
# conmakehash: Create arrays for initializing the kernel console tables
+PKG_CONFIG?= pkg-config
+
HOST_EXTRACFLAGS += -I$(srctree)/tools/include
+CRYPTO_LIBS = $(shell $(PKG_CONFIG) --libs libcrypto 2> /dev/null || -lcrypto)
+CRYPTO_CFLAGS = $(shell $(PKG_CONFIG) --cflags libcrypto 2> /dev/null)
hostprogs-$(CONFIG_BUILD_BIN2C) += bin2c
hostprogs-$(CONFIG_KALLSYMS) += kallsyms
@@ -23,8 +27,9 @@ hostprogs-$(CONFIG_SYSTEM_EXTRA_CERTIFICATE) += insert-sys-cert
HOSTCFLAGS_sortextable.o = -I$(srctree)/tools/include
HOSTCFLAGS_asn1_compiler.o = -I$(srctree)/include
-HOSTLDLIBS_sign-file = -lcrypto
-HOSTLDLIBS_extract-cert = -lcrypto
+HOSTLDLIBS_sign-file = $(CRYPTO_LIBS)
+HOSTCFLAGS_extract-cert.o = $(CRYPTO_CFLAGS)
+HOSTLDLIBS_extract-cert = $(CRYPTO_LIBS)
always := $(hostprogs-y) $(hostprogs-m)
--
2.19.1
--
Rolf Eike Beer, emlix GmbH, http://www.emlix.com
Fon +49 551 30664-0, Fax +49 551 30664-11
Gothaer Platz 3, 37083 Göttingen, Germany
Sitz der Gesellschaft: Göttingen, Amtsgericht Göttingen HR B 3160
Geschäftsführung: Heike Jordan, Dr. Uwe Kracke – Ust-IdNr.: DE 205 198 055
emlix - smart embedded open source
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2] scripts: use pkg-config to locate libcrypto
[not found] ` <20181124055339.E3B9120881@mail.kernel.org>
@ 2018-11-26 9:13 ` Rolf Eike Beer
2018-12-14 12:08 ` Rolf Eike Beer
0 siblings, 1 reply; 8+ messages in thread
From: Rolf Eike Beer @ 2018-11-26 9:13 UTC (permalink / raw)
To: Sasha Levin; +Cc: Linux Kernel Developers List
[-- Attachment #1: Type: text/plain, Size: 1152 bytes --]
Am Samstag, 24. November 2018, 06:53:39 CET schrieb Sasha Levin:
> Hi,
>
> [This is an automated email]
>
> This commit has been processed because it contains a -stable tag.
> The stable tag indicates that it's relevant for the following trees: all
>
> The bot has tested the following trees: v4.19.3, v4.14.82, v4.9.138,
> v4.4.164, v3.18.126.
>
> v4.19.3: Build failed! Errors:
> collect2: error: ld returned 1 exit status
This sounds like it could actually be a problem, can you point me to the full
log, please?
> v4.14.82: Failed to apply! Possible dependencies:
> Unable to calculate
This would either need 8377bd2b9ee1be35b39b5523f640a2b75ddd7c4e as well, or a
modified patch. Manually fixing this up should be trivial. Just tell me which
way you prefer.
Greetings,
Eike
--
Rolf Eike Beer, emlix GmbH, http://www.emlix.com
Fon +49 551 30664-0, Fax +49 551 30664-11
Gothaer Platz 3, 37083 Göttingen, Germany
Sitz der Gesellschaft: Göttingen, Amtsgericht Göttingen HR B 3160
Geschäftsführung: Heike Jordan, Dr. Uwe Kracke – Ust-IdNr.: DE 205 198 055
emlix - smart embedded open source
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 313 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2] scripts: use pkg-config to locate libcrypto
2018-11-26 9:13 ` Rolf Eike Beer
@ 2018-12-14 12:08 ` Rolf Eike Beer
0 siblings, 0 replies; 8+ messages in thread
From: Rolf Eike Beer @ 2018-12-14 12:08 UTC (permalink / raw)
To: Sasha Levin; +Cc: Linux Kernel Developers List, David Woodhouse
[-- Attachment #1: Type: text/plain, Size: 1250 bytes --]
Am Montag, 26. November 2018, 10:13:23 CET schrieb Rolf Eike Beer:
> Am Samstag, 24. November 2018, 06:53:39 CET schrieb Sasha Levin:
> > Hi,
> >
> > [This is an automated email]
> >
> > This commit has been processed because it contains a -stable tag.
> > The stable tag indicates that it's relevant for the following trees: all
> >
> > The bot has tested the following trees: v4.19.3, v4.14.82, v4.9.138,
> > v4.4.164, v3.18.126.
> >
> > v4.19.3: Build failed! Errors:
> > collect2: error: ld returned 1 exit status
>
> This sounds like it could actually be a problem, can you point me to the
> full log, please?
>
> > v4.14.82: Failed to apply! Possible dependencies:
> > Unable to calculate
>
> This would either need 8377bd2b9ee1be35b39b5523f640a2b75ddd7c4e as well, or
> a modified patch. Manually fixing this up should be trivial. Just tell me
> which way you prefer.
Ping?
--
Rolf Eike Beer, emlix GmbH, http://www.emlix.com
Fon +49 551 30664-0, Fax +49 551 30664-11
Gothaer Platz 3, 37083 Göttingen, Germany
Sitz der Gesellschaft: Göttingen, Amtsgericht Göttingen HR B 3160
Geschäftsführung: Heike Jordan, Dr. Uwe Kracke – Ust-IdNr.: DE 205 198 055
emlix - smart embedded open source
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 313 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 RESEND] scripts: use pkg-config to locate libcrypto
2018-11-22 16:42 ` [PATCH v2] " Rolf Eike Beer
[not found] ` <20181124055339.E3B9120881@mail.kernel.org>
@ 2019-06-06 7:55 ` Rolf Eike Beer
2019-06-06 8:17 ` David Woodhouse
1 sibling, 1 reply; 8+ messages in thread
From: Rolf Eike Beer @ 2019-06-06 7:55 UTC (permalink / raw)
To: David Woodhouse; +Cc: Linux Kernel Developers List, David Howells, keyrings
From cca931322233827dc21c7609f21f4042d78f220e Mon Sep 17 00:00:00 2001
From: Rolf Eike Beer <eb@emlix.com>
Date: Thu, 22 Nov 2018 16:40:49 +0100
Subject: scripts: use pkg-config to locate libcrypto
Otherwise build fails if the headers are not in the default location. While at
it also ask pkg-config for the libs, with fallback to the existing value.
Signed-off-by: Rolf Eike Beer <eb@emlix.com>
Cc: stable@vger.kernel.org # 4.19.x
---
scripts/Makefile | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
Last time I got notice about a build error with 4.19.3, but it works fine for
me on top of both 4.19 and 4.19.48.
diff --git a/scripts/Makefile b/scripts/Makefile
index 9d442ee050bd..bd2a30b43f28 100644
--- a/scripts/Makefile
+++ b/scripts/Makefile
@@ -8,7 +8,11 @@
# conmakehash: Create chartable
# conmakehash: Create arrays for initializing the kernel console tables
+PKG_CONFIG?= pkg-config
+
HOST_EXTRACFLAGS += -I$(srctree)/tools/include
+CRYPTO_LIBS = $(shell $(PKG_CONFIG) --libs libcrypto 2> /dev/null || -lcrypto)
+CRYPTO_CFLAGS = $(shell $(PKG_CONFIG) --cflags libcrypto 2> /dev/null)
hostprogs-$(CONFIG_BUILD_BIN2C) += bin2c
hostprogs-$(CONFIG_KALLSYMS) += kallsyms
@@ -23,8 +27,9 @@ hostprogs-$(CONFIG_SYSTEM_EXTRA_CERTIFICATE) += insert-sys-cert
HOSTCFLAGS_sortextable.o = -I$(srctree)/tools/include
HOSTCFLAGS_asn1_compiler.o = -I$(srctree)/include
-HOSTLDLIBS_sign-file = -lcrypto
-HOSTLDLIBS_extract-cert = -lcrypto
+HOSTLDLIBS_sign-file = $(CRYPTO_LIBS)
+HOSTCFLAGS_extract-cert.o = $(CRYPTO_CFLAGS)
+HOSTLDLIBS_extract-cert = $(CRYPTO_LIBS)
always := $(hostprogs-y) $(hostprogs-m)
--
2.21.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2 RESEND] scripts: use pkg-config to locate libcrypto
2019-06-06 7:55 ` [PATCH v2 RESEND] " Rolf Eike Beer
@ 2019-06-06 8:17 ` David Woodhouse
2019-06-06 8:45 ` Rolf Eike Beer
0 siblings, 1 reply; 8+ messages in thread
From: David Woodhouse @ 2019-06-06 8:17 UTC (permalink / raw)
To: Rolf Eike Beer; +Cc: Linux Kernel Developers List, David Howells, keyrings
[-- Attachment #1: Type: text/plain, Size: 417 bytes --]
On Thu, 2019-06-06 at 09:55 +0200, Rolf Eike Beer wrote:
> +CRYPTO_LIBS = $(shell $(PKG_CONFIG) --libs libcrypto 2> /dev/null || -lcrypto)
That's going to run:
$ pkg-config --libs libcrypto || -lcrypto
If libcrypto.pc isn't there, it's going to get this:
-lcrypto: command not found
I think you meant:
+CRYPTO_LIBS = $(shell $(PKG_CONFIG) --libs libcrypto 2> /dev/null || echo -lcrypto)
[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 5174 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2 RESEND] scripts: use pkg-config to locate libcrypto
2019-06-06 8:17 ` David Woodhouse
@ 2019-06-06 8:45 ` Rolf Eike Beer
0 siblings, 0 replies; 8+ messages in thread
From: Rolf Eike Beer @ 2019-06-06 8:45 UTC (permalink / raw)
To: David Woodhouse; +Cc: Linux Kernel Developers List, David Howells, keyrings
[-- Attachment #1: Type: text/plain, Size: 838 bytes --]
David Woodhouse wrote:
> On Thu, 2019-06-06 at 09:55 +0200, Rolf Eike Beer wrote:
> > +CRYPTO_LIBS = $(shell $(PKG_CONFIG) --libs libcrypto 2> /dev/null ||
> > -lcrypto)
> That's going to run:
>
> $ pkg-config --libs libcrypto || -lcrypto
>
>
> If libcrypto.pc isn't there, it's going to get this:
>
>
> -lcrypto: command not found
>
> I think you meant:
>
> +CRYPTO_LIBS = $(shell $(PKG_CONFIG) --libs libcrypto 2> /dev/null || echo
> -lcrypto)
Doh! Thanks, v3 in a minute.
Eike
--
Rolf Eike Beer, emlix GmbH, http://www.emlix.com
Fon +49 551 30664-0, Fax +49 551 30664-11
Gothaer Platz 3, 37083 Göttingen, Germany
Sitz der Gesellschaft: Göttingen, Amtsgericht Göttingen HR B 3160
Geschäftsführung: Heike Jordan, Dr. Uwe Kracke – Ust-IdNr.: DE 205 198 055
emlix - smart embedded open source
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 313 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-06-06 8:45 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-22 15:45 [PATCH] scripts: use pkg-config to locate libcrypto Rolf Eike Beer
2018-11-22 16:16 ` David Woodhouse
2018-11-22 16:42 ` [PATCH v2] " Rolf Eike Beer
[not found] ` <20181124055339.E3B9120881@mail.kernel.org>
2018-11-26 9:13 ` Rolf Eike Beer
2018-12-14 12:08 ` Rolf Eike Beer
2019-06-06 7:55 ` [PATCH v2 RESEND] " Rolf Eike Beer
2019-06-06 8:17 ` David Woodhouse
2019-06-06 8:45 ` Rolf Eike Beer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).