From: Peter Korsgaard <peter@korsgaard.com>
To: <yann.morin@orange.com>
Cc: Maxim Kochetkov <fido_max@inbox.ru>,
Bernd Kuhls <bernd@kuhls.net>,
buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH v2] package/erlang: disable for uclibc, fix glibc-build
Date: Fri, 01 Dec 2023 21:00:33 +0100 [thread overview]
Message-ID: <87msut66ce.fsf@48ers.dk> (raw)
In-Reply-To: <5e5da455b1f21f427b740a6cc9206c3b58316a9c.1701078039.git.yann.morin@orange.com> (yann morin's message of "Mon, 27 Nov 2023 10:40:44 +0100")
>>>>> <yann.morin@orange.com> writes:
> From: "Yann E. MORIN" <yann.morin@orange.com>
> Commit 2cfa86a54882(package/erlang: bump version to 26.0.2) added a
> patch to restore building on uClibc.
> However, that patch is not upstream, and has been rejected:
> https://github.com/erlang/otp/pull/7500
> Please open a PR to https://github.com/asmjit/asmjit instead and we
> will get the fix next time we sync with upstream. We do not want
> theirs and our implementation to diverge.
> Furthermore, it happens to work on uClibc, because uClibc does not
> expose sys/auxv.h, but it fails to work on glibc, because the define is
> not propagated to "sub-trees", and thus is never defined where it is
> checked for, even when sys/auxv.h is available. This causes build
> failures such as:
> asmjit/core/cpuinfo.cpp: In function ‘void asmjit::_abi_1_10::detectHWCaps(CpuInfo&, long unsigned int, const LinuxHWCapMapping*, size_t)’:
> asmjit/core/cpuinfo.cpp:840:24: error: ‘getauxval’ was not declared in this scope
> 840 | unsigned long mask = getauxval(type);
> | ^~~~~~~~~
> asmjit/core/cpuinfo.cpp: In function ‘void asmjit::_abi_1_10::detectARMCpu(CpuInfo&)’:
> asmjit/core/cpuinfo.cpp:972:21: error: ‘AT_HWCAP’ was not declared in this scope
> 972 | detectHWCaps(cpu, AT_HWCAP, hwCapMapping, ASMJIT_ARRAY_SIZE(hwCapMapping));
> | ^~~~~~~~
> asmjit/core/cpuinfo.cpp:973:21: error: ‘AT_HWCAP2’ was not declared in this scope
> 973 | detectHWCaps(cpu, AT_HWCAP2, hwCapMapping2, ASMJIT_ARRAY_SIZE(hwCapMapping2));
> | ^~~~~~~~~
> Yet, sys/auxv.h was detected at configure time:
> checking for sys/auxv.h... yes
> This defconfig is enough to reproduce the error:
> BR2_aarch64=y
> BR2_TOOLCHAIN_EXTERNAL=y
> BR2_TOOLCHAIN_EXTERNAL_BOOTLIN=y
> BR2_PACKAGE_ERLANG=y
> Since upstream refused the patch, and there is no fix that was submitted
> to the actual upstream (asmjit), drop the rejectred patch, and disable
> for uClibc: the patch is incorrect, and we can't fix a build issue on
> uClibc by introducing another on glibc.
> Fixes:
> http://autobuild.buildroot.org/results/fc1/fc19bad2263bdfacea594217d5ddfde0e27895b1/
> http://autobuild.buildroot.org/results/114/11416d81d5b27fc0627b335a971154c088d5754a/
> Signed-off-by: Yann E. MORIN <yann.morin@orange.com>
> Cc: Bernd Kuhls <bernd@kuhls.net>
> Cc: Maxim Kochetkov <fido_max@inbox.ru>
> Changes v1 -> v2:
> - update comment when unavailable
Committed to 2023.08.x, thanks.
--
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
prev parent reply other threads:[~2023-12-01 20:00 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-27 9:40 [Buildroot] [PATCH v2] package/erlang: disable for uclibc, fix glibc-build yann.morin
2023-11-29 7:41 ` Peter Korsgaard
2023-12-01 20:00 ` Peter Korsgaard [this message]
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=87msut66ce.fsf@48ers.dk \
--to=peter@korsgaard.com \
--cc=bernd@kuhls.net \
--cc=buildroot@buildroot.org \
--cc=fido_max@inbox.ru \
--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 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.