* [Buildroot] [PATCH 1/1] package/tor: Fix build with libressl >= 3.8.1
@ 2024-01-28 16:41 Bernd Kuhls
2024-01-28 17:51 ` Yann E. MORIN
2024-02-28 17:35 ` Peter Korsgaard
0 siblings, 2 replies; 3+ messages in thread
From: Bernd Kuhls @ 2024-01-28 16:41 UTC (permalink / raw)
To: buildroot
Fixes:
http://autobuild.buildroot.net/results/85c/85cde3bcd12fb5adafb94c85d5fa636e1b5b9068/
Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
---
...t-define-DISABLE_ENGINES-after-OPENS.patch | 61 +++++++++++++++++++
1 file changed, 61 insertions(+)
create mode 100644 package/tor/0001-crypt_openssl_mgt-define-DISABLE_ENGINES-after-OPENS.patch
diff --git a/package/tor/0001-crypt_openssl_mgt-define-DISABLE_ENGINES-after-OPENS.patch b/package/tor/0001-crypt_openssl_mgt-define-DISABLE_ENGINES-after-OPENS.patch
new file mode 100644
index 0000000000..f46acb28c8
--- /dev/null
+++ b/package/tor/0001-crypt_openssl_mgt-define-DISABLE_ENGINES-after-OPENS.patch
@@ -0,0 +1,61 @@
+From 48f8d6918977673125d53a85d19d709136106739 Mon Sep 17 00:00:00 2001
+From: orbea <orbea@riseup.net>
+Date: Thu, 31 Aug 2023 14:35:52 -0700
+Subject: [PATCH] crypt_openssl_mgt: define DISABLE_ENGINES after
+ OPENSSL_NO_ENGINE
+
+With LibreSSL-3.8.1 these engines are no long available causing a build
+failure, but LibreSSL correctly defines OPENSSL_NO_ENGINE as part of its
+opensslfeatures.h. However Tor includes crypto_openssl_mgt.h before any
+of the openssl includes which would define OPENSSL_NO_ENGINE and then
+fails to define DISABLE_ENGINES.
+
+As the define is used in only a single .c file it is best to move it
+there.
+
+Signed-off-by: orbea <orbea@riseup.net>
+
+Upstream:
+https://gitlab.torproject.org/tpo/core/tor/-/commit/48f8d6918977673125d53a85d19d709136106739
+
+Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
+---
+ src/lib/crypt_ops/crypto_openssl_mgt.c | 5 +++++
+ src/lib/crypt_ops/crypto_openssl_mgt.h | 5 -----
+ 2 files changed, 5 insertions(+), 5 deletions(-)
+
+diff --git a/src/lib/crypt_ops/crypto_openssl_mgt.c b/src/lib/crypt_ops/crypto_openssl_mgt.c
+index 6c01cb6aa8..ca12a82518 100644
+--- a/src/lib/crypt_ops/crypto_openssl_mgt.c
++++ b/src/lib/crypt_ops/crypto_openssl_mgt.c
+@@ -40,6 +40,11 @@ ENABLE_GCC_WARNING("-Wredundant-decls")
+
+ #include <string.h>
+
++#ifdef OPENSSL_NO_ENGINE
++/* Android's OpenSSL seems to have removed all of its Engine support. */
++#define DISABLE_ENGINES
++#endif
++
+ #ifndef NEW_THREAD_API
+ /** A number of preallocated mutexes for use by OpenSSL. */
+ static tor_mutex_t **openssl_mutexes_ = NULL;
+diff --git a/src/lib/crypt_ops/crypto_openssl_mgt.h b/src/lib/crypt_ops/crypto_openssl_mgt.h
+index 96a37721dd..eac0ec1977 100644
+--- a/src/lib/crypt_ops/crypto_openssl_mgt.h
++++ b/src/lib/crypt_ops/crypto_openssl_mgt.h
+@@ -49,11 +49,6 @@
+ #define OPENSSL_V_SERIES(a,b,c) \
+ OPENSSL_VER((a),(b),(c),0,0)
+
+-#ifdef OPENSSL_NO_ENGINE
+-/* Android's OpenSSL seems to have removed all of its Engine support. */
+-#define DISABLE_ENGINES
+-#endif
+-
+ #if OPENSSL_VERSION_NUMBER >= OPENSSL_VER(1,1,0,0,5)
+ /* OpenSSL as of 1.1.0pre4 has an "new" thread API, which doesn't require
+ * setting up various callbacks.
+--
+GitLab
+
--
2.39.2
_______________________________________________
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/tor: Fix build with libressl >= 3.8.1
2024-01-28 16:41 [Buildroot] [PATCH 1/1] package/tor: Fix build with libressl >= 3.8.1 Bernd Kuhls
@ 2024-01-28 17:51 ` Yann E. MORIN
2024-02-28 17:35 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Yann E. MORIN @ 2024-01-28 17:51 UTC (permalink / raw)
To: Bernd Kuhls; +Cc: buildroot
Bernd, All,
On 2024-01-28 17:41 +0100, Bernd Kuhls spake thusly:
> Fixes:
> http://autobuild.buildroot.net/results/85c/85cde3bcd12fb5adafb94c85d5fa636e1b5b9068/
>
> Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
> ---
> ...t-define-DISABLE_ENGINES-after-OPENS.patch | 61 +++++++++++++++++++
> 1 file changed, 61 insertions(+)
> create mode 100644 package/tor/0001-crypt_openssl_mgt-define-DISABLE_ENGINES-after-OPENS.patch
>
> diff --git a/package/tor/0001-crypt_openssl_mgt-define-DISABLE_ENGINES-after-OPENS.patch b/package/tor/0001-crypt_openssl_mgt-define-DISABLE_ENGINES-after-OPENS.patch
> new file mode 100644
> index 0000000000..f46acb28c8
> --- /dev/null
> +++ b/package/tor/0001-crypt_openssl_mgt-define-DISABLE_ENGINES-after-OPENS.patch
> @@ -0,0 +1,61 @@
> +From 48f8d6918977673125d53a85d19d709136106739 Mon Sep 17 00:00:00 2001
> +From: orbea <orbea@riseup.net>
> +Date: Thu, 31 Aug 2023 14:35:52 -0700
> +Subject: [PATCH] crypt_openssl_mgt: define DISABLE_ENGINES after
> + OPENSSL_NO_ENGINE
> +
> +With LibreSSL-3.8.1 these engines are no long available causing a build
> +failure, but LibreSSL correctly defines OPENSSL_NO_ENGINE as part of its
> +opensslfeatures.h. However Tor includes crypto_openssl_mgt.h before any
> +of the openssl includes which would define OPENSSL_NO_ENGINE and then
> +fails to define DISABLE_ENGINES.
> +
> +As the define is used in only a single .c file it is best to move it
> +there.
> +
> +Signed-off-by: orbea <orbea@riseup.net>
> +
> +Upstream:
> +https://gitlab.torproject.org/tpo/core/tor/-/commit/48f8d6918977673125d53a85d19d709136106739
$ make check-package
package/tor/0001-crypt_openssl_mgt-define-DISABLE_ENGINES-after-OPENS.patch:0: missing Upstream in the header (http://nightly.buildroot.org/#_additional_patch_documentation)
Indeed, the upstream status must be on the same line as the Upstream
tag. I've fixed that and squashed the empty lines around it (no need
to waste too much space on the chain of custody).
Applied to master, thanks.
Regards,
Yann E. MORIN.
> +Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
> +---
> + src/lib/crypt_ops/crypto_openssl_mgt.c | 5 +++++
> + src/lib/crypt_ops/crypto_openssl_mgt.h | 5 -----
> + 2 files changed, 5 insertions(+), 5 deletions(-)
> +
> +diff --git a/src/lib/crypt_ops/crypto_openssl_mgt.c b/src/lib/crypt_ops/crypto_openssl_mgt.c
> +index 6c01cb6aa8..ca12a82518 100644
> +--- a/src/lib/crypt_ops/crypto_openssl_mgt.c
> ++++ b/src/lib/crypt_ops/crypto_openssl_mgt.c
> +@@ -40,6 +40,11 @@ ENABLE_GCC_WARNING("-Wredundant-decls")
> +
> + #include <string.h>
> +
> ++#ifdef OPENSSL_NO_ENGINE
> ++/* Android's OpenSSL seems to have removed all of its Engine support. */
> ++#define DISABLE_ENGINES
> ++#endif
> ++
> + #ifndef NEW_THREAD_API
> + /** A number of preallocated mutexes for use by OpenSSL. */
> + static tor_mutex_t **openssl_mutexes_ = NULL;
> +diff --git a/src/lib/crypt_ops/crypto_openssl_mgt.h b/src/lib/crypt_ops/crypto_openssl_mgt.h
> +index 96a37721dd..eac0ec1977 100644
> +--- a/src/lib/crypt_ops/crypto_openssl_mgt.h
> ++++ b/src/lib/crypt_ops/crypto_openssl_mgt.h
> +@@ -49,11 +49,6 @@
> + #define OPENSSL_V_SERIES(a,b,c) \
> + OPENSSL_VER((a),(b),(c),0,0)
> +
> +-#ifdef OPENSSL_NO_ENGINE
> +-/* Android's OpenSSL seems to have removed all of its Engine support. */
> +-#define DISABLE_ENGINES
> +-#endif
> +-
> + #if OPENSSL_VERSION_NUMBER >= OPENSSL_VER(1,1,0,0,5)
> + /* OpenSSL as of 1.1.0pre4 has an "new" thread API, which doesn't require
> + * setting up various callbacks.
> +--
> +GitLab
> +
> --
> 2.39.2
>
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
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/tor: Fix build with libressl >= 3.8.1
2024-01-28 16:41 [Buildroot] [PATCH 1/1] package/tor: Fix build with libressl >= 3.8.1 Bernd Kuhls
2024-01-28 17:51 ` Yann E. MORIN
@ 2024-02-28 17:35 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2024-02-28 17:35 UTC (permalink / raw)
To: Bernd Kuhls; +Cc: buildroot
>>>>> "Bernd" == Bernd Kuhls <bernd@kuhls.net> writes:
> Fixes:
> http://autobuild.buildroot.net/results/85c/85cde3bcd12fb5adafb94c85d5fa636e1b5b9068/
> Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
Committed to 2023.11.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:[~2024-02-28 17:35 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-28 16:41 [Buildroot] [PATCH 1/1] package/tor: Fix build with libressl >= 3.8.1 Bernd Kuhls
2024-01-28 17:51 ` Yann E. MORIN
2024-02-28 17:35 ` Peter Korsgaard
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox