[Buildroot] [PATCH] package/libssh: select BR2_PACKAGE_LIBOPENSSL_ENGINES

[Buildroot] [PATCH] package/libssh: select BR2_PACKAGE_LIBOPENSSL_ENGINES

[Heiko Thiery]

This fixes link time errors, undefined references to various ENGINE_*
functions in case libopenssl backend is used.

Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com>
---
 package/libssh/Config.in | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/libssh/Config.in b/package/libssh/Config.in
index 1f271a692e..eaec754414 100644
--- a/package/libssh/Config.in
+++ b/package/libssh/Config.in
@@ -36,6 +36,7 @@ config BR2_PACKAGE_LIBSSH_LIBGCRYPT
 config BR2_PACKAGE_LIBSSH_OPENSSL
 	bool "openssl"
 	depends on BR2_PACKAGE_OPENSSL
+	select BR2_PACKAGE_LIBOPENSSL_ENGINES
 
 endchoice
 
-- 
2.20.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] package/libssh: select BR2_PACKAGE_LIBOPENSSL_ENGINES

[Thomas Petazzoni via buildroot]

On Tue,  9 Jul 2024 22:24:39 +0200
Heiko Thiery <heiko.thiery@gmail.com> wrote:

> This fixes link time errors, undefined references to various ENGINE_*
> functions in case libopenssl backend is used.
> 
> Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com>

Is this
fixing http://autobuild.buildroot.net/results/5bb/5bb845be7f861dc3540a8dc618ccd1a0a759f2d5/build-end.log
and similar build issues ?

Thanks!

Thomas
-- 
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

Re: [Buildroot] [PATCH] package/libssh: select BR2_PACKAGE_LIBOPENSSL_ENGINES

[Heiko Thiery]

Hi,

Am Mi., 10. Juli 2024 um 16:51 Uhr schrieb Thomas Petazzoni
<thomas.petazzoni@bootlin.com>:
>
> On Tue,  9 Jul 2024 22:24:39 +0200
> Heiko Thiery <heiko.thiery@gmail.com> wrote:
>
> > This fixes link time errors, undefined references to various ENGINE_*
> > functions in case libopenssl backend is used.
> >
> > Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com>
>
> Is this
> fixing http://autobuild.buildroot.net/results/5bb/5bb845be7f861dc3540a8dc618ccd1a0a759f2d5/build-end.log
> and similar build issues ?

Yes this looks like the issue I had.

-- 
Heiko

>
> Thanks!
>
> Thomas
> --
> 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

Re: [Buildroot] [PATCH] package/libssh: select BR2_PACKAGE_LIBOPENSSL_ENGINES

[Thomas Petazzoni via buildroot]

On Tue,  9 Jul 2024 22:24:39 +0200
Heiko Thiery <heiko.thiery@gmail.com> wrote:

> This fixes link time errors, undefined references to various ENGINE_*
> functions in case libopenssl backend is used.
> 
> Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com>
> ---
>  package/libssh/Config.in | 1 +
>  1 file changed, 1 insertion(+)

Applied to master, after adding a reference to the autobuilder build
failure. Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] package/libssh: select BR2_PACKAGE_LIBOPENSSL_ENGINES

[Joachim Wiberg]

On Wed, 2024-07-10 at 16:51 +0200, Thomas Petazzoni via buildroot
wrote:
> On Tue,  9 Jul 2024 22:24:39 +0200
> Heiko Thiery <heiko.thiery@gmail.com> wrote:
> > This fixes link time errors, undefined references to various
> > ENGINE_* functions in case libopenssl backend is used.
> > 
> > Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com>
> 
> Is this fixing
> http://autobuild.buildroot.net/results/5bb/5bb845be7f861dc3540a8dc618ccd1a0a759f2d5/build-end.log
> and similar build issues ?
> 

Indeed it is.   Same issue also on 2024.02.x (ping Peter)

It seems to have accidentally worked after the upgrade from OpenSSL 1.x
to 3.x, but broke either in 8fed480 or 1899589 (also on 2024.02.x)

Tested-by: Joachim Wiberg <troglobit@gmail.com>

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] package/libssh: select BR2_PACKAGE_LIBOPENSSL_ENGINES

[Joachim Wiberg]

Hi,

just a friendly reminder.  Maybe my initial reply got lost in all the
noise and summer vacation.

On 2024-07-10 at 23:13 +0200, Thomas Petazzoni wrote:
> Heiko Thiery <heiko.thiery@gmail.com> wrote:
> > This fixes link time errors, undefined references to various
> > ENGINE_* functions in case libopenssl backend is used.
> Applied to master, after adding a reference to the autobuilder build
> failure. Thanks!

We have the same issue also on 2024.02.x, as of 2024.02.4 builds with
libssh break unless you update your defconfig.  So it would be great if
62103be918e7a155749b1b3b680a9e4ecba6af87 could be cherry picked for the
next LTS patch release.

Best regards
 /Joachim

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] package/libssh: select BR2_PACKAGE_LIBOPENSSL_ENGINES

[Baruch Siach via buildroot]

Hi Joachim,

On Tue, Jul 16 2024, Joachim Wiberg wrote:
> On 2024-07-10 at 23:13 +0200, Thomas Petazzoni wrote:
>> Heiko Thiery <heiko.thiery@gmail.com> wrote:
>> > This fixes link time errors, undefined references to various
>> > ENGINE_* functions in case libopenssl backend is used.
>> Applied to master, after adding a reference to the autobuilder build
>> failure. Thanks!
>
> We have the same issue also on 2024.02.x, as of 2024.02.4 builds with
> libssh break unless you update your defconfig.  So it would be great if
> 62103be918e7a155749b1b3b680a9e4ecba6af87 could be cherry picked for the
> next LTS patch release.

Commit fed58dac5dea (Revert "package/libopenssl: no-engine") should have
fixed that in 2024.02.4. Are you sure libssh build is still broken?

baruch

-- 
                                                     ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch@tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] package/libssh: select BR2_PACKAGE_LIBOPENSSL_ENGINES

[Joachim Wiberg]

Hi Baruch,

On Tue, 2024-07-16 at 12:21 +0300, Baruch Siach wrote:
> Hi Joachim,
> 
> On Tue, Jul 16 2024, Joachim Wiberg wrote:
> > On 2024-07-10 at 23:13 +0200, Thomas Petazzoni wrote:
> > > Applied to master, after adding a reference to the autobuilder
> > > build failure. Thanks!
> > We have the same issue also on 2024.02.x, as of 2024.02.4 builds
> > with libssh break unless you update your defconfig.
> Commit fed58dac5dea (Revert "package/libopenssl: no-engine") should
> have fixed that in 2024.02.4. Are you sure libssh build is still
> broken?

Just retested, and you are 100% correct, the libssh build on 2024.02.x
is *not broken*.

So sorry having wasted everyone's time!

All the best
 /Joachim

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot