Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Seiderer via buildroot <buildroot@buildroot.org>
To: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: buildroot@busybox.net, yann.morin@orange.com,
	Samuel Martin <s.martin49@gmail.com>
Subject: Re: [Buildroot] [PATCH v2 5/5] package/xz: convert to cmake build
Date: Thu, 27 Jun 2024 10:26:06 +0200	[thread overview]
Message-ID: <20240627102606.7de6b9cd@gmx.net> (raw)
In-Reply-To: <20240627095029.2d1fa5c0@gmx.net>

On Thu, 27 Jun 2024 09:50:29 +0200, Peter Seiderer <ps.report@gmx.net> wrote:

[...]

> >
> > Anyway, I have now reverted that patch, as it was exposing another issue
> > with host-python3 (https://gitlab.com/buildroot.org/buildroot/-/issues/5)
>
> Yes, reverting the patch seems to be the best solution for the moment...

The following quick patch solves the host-xz and host-python3 build failures
for me:

diff --git a/package/xz/xz.mk b/package/xz/xz.mk
index b22b96849d..b22ccac85f 100644
--- a/package/xz/xz.mk
+++ b/package/xz/xz.mk
@@ -12,7 +12,7 @@ XZ_LICENSE = Public Domain, BSD-0-Clause, GPL-2.0+, GPL-3.0+, LGPL-2.1+
 XZ_LICENSE_FILES = COPYING COPYING.0BSD COPYING.GPLv2 COPYING.GPLv3 COPYING.LGPLv2.1
 XZ_CPE_ID_VENDOR = tukaani

-XZ_CONF_OPTS = \
+XZ_COMMON_CONF_OPTS = \
 	-DENCODERS="lzma1;lzma2;delta;x86;powerpc;ia64;arm;armthumb;arm64;sparc;riscv" \
 	-DDECODERS="lzma1;lzma2;delta;x86;powerpc;ia64;arm;armthumb;arm64;sparc;riscv" \
 	-DMATCH_FINDERS="hc3;hc4;bt2;bt3;bt4" \
@@ -25,8 +25,9 @@ XZ_CONF_OPTS = \
 	-DENABLE_SMALL=OFF \
 	-DENABLE_SANDBOX=ON \
 	-DCREATE_XZ_SYMLINKS=ON \
-	-DCREATE_LZMA_SYMLINKS=ON \
-	-DBUILD_SHARED_LIBS=OFF
+	-DCREATE_LZMA_SYMLINKS=ON
+
+XZ_CONF_OPTS = $(XZ_COMMON_CONF_OPTS)

 ifeq ($(BR2_SYSTEM_ENABLE_NLS),y)
 XZ_CONF_OPTS += -DENABLE_NLS=ON
@@ -42,11 +43,11 @@ endif

 # we are built before ccache
 HOST_XZ_CONF_OPTS = \
-	$(XZ_CONF_OPTS) \
-	-DENABLE_NLS=ON \
+	$(XZ_COMMON_CONF_OPTS) \
+	-DENABLE_NLS=OFF \
 	-DENABLE_THREADS=ON \
 	-DCMAKE_C_COMPILER_LAUNCHER="" \
-	-DCMAKE_CXX_COMPILER_LAUNCHER==""
+	-DCMAKE_CXX_COMPILER_LAUNCHER=""

 $(eval $(cmake-package))
 $(eval $(host-cmake-package))

Regards,
Peter


>
> Maybe although reverting 'package/xz: explicitly specify all autoconf options'
> (and 'package/xz: enable year2038 option') as I used the same
> 'HOST_XZ_CONF_OPTS = $(XZ_CONF_OPTS)...' pattern here?
>
> Regards,
> Peter
>

[...]

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

  parent reply	other threads:[~2024-06-27  8:26 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-06-12 13:57 [Buildroot] [PATCH v2 1/5] package/xz: bump version to 5.4.7 Peter Seiderer via buildroot
2024-06-12 13:57 ` [Buildroot] [PATCH v2 2/5] package/xz: bump version to 5.6.2 Peter Seiderer via buildroot
2024-06-12 15:48   ` Julien Olivain
2024-06-24 13:42   ` Arnout Vandecappelle via buildroot
2024-06-12 13:57 ` [Buildroot] [PATCH v2 3/5] package/xz: determine all autoconf options Peter Seiderer via buildroot
2024-06-24 13:44   ` Arnout Vandecappelle via buildroot
2024-06-12 13:57 ` [Buildroot] [PATCH v2 4/5] package/xz: enable year2038 option Peter Seiderer via buildroot
2024-06-24 13:46   ` Arnout Vandecappelle via buildroot
2024-06-12 13:57 ` [Buildroot] [PATCH v2 5/5] package/xz: convert to cmake build Peter Seiderer via buildroot
2024-06-24 13:52   ` Arnout Vandecappelle via buildroot
2024-06-25  9:56   ` yann.morin
2024-06-25 11:11     ` yann.morin
2024-06-26  8:36       ` Peter Seiderer via buildroot
2024-06-26 19:32         ` Yann E. MORIN
2024-06-27  7:50           ` Peter Seiderer via buildroot
2024-06-27  7:57             ` Peter Seiderer via buildroot
2024-06-27  8:26             ` Peter Seiderer via buildroot [this message]
2024-06-27 11:16               ` yann.morin
2024-07-02 12:47                 ` Peter Seiderer via buildroot
2024-06-24 13:41 ` [Buildroot] [PATCH v2 1/5] package/xz: bump version to 5.4.7 Arnout Vandecappelle via buildroot
2024-07-08 10:04   ` Peter Korsgaard
2024-07-08 12:54     ` Peter Seiderer via buildroot
2024-07-08 12:57       ` Peter Seiderer via buildroot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240627102606.7de6b9cd@gmx.net \
    --to=buildroot@buildroot.org \
    --cc=buildroot@busybox.net \
    --cc=ps.report@gmx.net \
    --cc=s.martin49@gmail.com \
    --cc=yann.morin.1998@free.fr \
    --cc=yann.morin@orange.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox