public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Michael Orlitzky" <mjo@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: dev-libs/ntl/
Date: Wed, 14 Jun 2023 11:15:54 +0000 (UTC)	[thread overview]
Message-ID: <1686741271.9ed8c182abf1232c57446017da57b24c4351a34c.mjo@gentoo> (raw)

commit:     9ed8c182abf1232c57446017da57b24c4351a34c
Author:     Michael Orlitzky <mjo <AT> gentoo <DOT> org>
AuthorDate: Wed Jun 14 06:26:26 2023 +0000
Commit:     Michael Orlitzky <mjo <AT> gentoo <DOT> org>
CommitDate: Wed Jun 14 11:14:31 2023 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=9ed8c182

dev-libs/ntl: another AVX FFT build fix, this time on amd64.

Even on amd64, detection of the CPU features needed for AVX FFT can
fail. In particular, it can fail when -march=native is absent from
CXXFLAGS. Since the upstream documentation calls the AVX FFT
experimental, we disable it entirely. We also bring back NATIVE=on,
which should somewhat hide the CXXFLAGS issue until it can be addressed
upstream.

Closes: https://bugs.gentoo.org/815775
Signed-off-by: Michael Orlitzky <mjo <AT> gentoo.org>

 dev-libs/ntl/ntl-11.5.1-r4.ebuild | 91 +++++++++++++++++++++++++++++++++++++++
 1 file changed, 91 insertions(+)

diff --git a/dev-libs/ntl/ntl-11.5.1-r4.ebuild b/dev-libs/ntl/ntl-11.5.1-r4.ebuild
new file mode 100644
index 000000000000..a7373655ffa1
--- /dev/null
+++ b/dev-libs/ntl/ntl-11.5.1-r4.ebuild
@@ -0,0 +1,91 @@
+# Copyright 1999-2023 Gentoo Authors
+# Distributed under the terms of the GNU General Public License v2
+
+EAPI=8
+
+inherit toolchain-funcs gnuconfig
+
+DESCRIPTION="High-performance and portable C++ number theory library"
+HOMEPAGE="https://www.shoup.net/ntl/ https://github.com/libntl/ntl"
+SRC_URI="https://www.shoup.net/ntl/${P}.tar.gz"
+
+LICENSE="LGPL-2.1+"
+SLOT="0/44"
+KEYWORDS="~amd64 ~arm64 ~loong ~ppc ~ppc64 ~riscv ~x86 ~amd64-linux ~x86-linux ~ppc-macos ~x64-macos"
+IUSE="doc threads"
+
+BDEPEND="dev-lang/perl"
+DEPEND="dev-libs/gmp:0=
+	dev-libs/gf2x
+	threads? ( >=dev-libs/gf2x-1.2 )"
+RDEPEND="${DEPEND}"
+
+S="${WORKDIR}/${P}/src"
+
+DOCS=( "${WORKDIR}/${P}"/README )
+
+src_unpack() {
+	default
+	gnuconfig_update "${S}/libtool-origin/"
+}
+
+src_configure() {
+	# The DoConfig script builds its own libtool, but doesn't
+	# really try to set up the build environment (bug 718892).
+	export CC="$(tc-getCC)"
+	export CXX="$(tc-getCXX)"
+
+	# Currently the build system can build a static library or both
+	# static and shared libraries, but not only shared libraries. The
+	# name NTL_GMP_LIP is *not* a typo.
+	#
+	# We have left NTL_ENABLE_AVX_FFT unconditionally disabled: NTL's
+	# AVX2 detection can fail even when the CPU supports it (bug
+	# 815775), and moreover, can fail due to CXXFLAGS. When that
+	# happens, and if we try to use the AVX FFT, the build fails.
+	# Finally, doc/config.txt says, "this is experimental at moment, and
+	# may lead to worse performance." So we are probably not missing out
+	# on much.
+	#
+	# The NATIVE=on option adds "-march=native" to CXXFLAGS and should
+	# not be enabled on Gentoo, but is currently necessary for NTL's CPU
+	# feature detection to work (bug 815775). See the upstream issue,
+	#
+	#  https://github.com/libntl/ntl/issues/22
+	#
+	perl DoConfig \
+		PREFIX="${EPREFIX}"/usr \
+		LIBDIR="${EPREFIX}"/usr/$(get_libdir) \
+		CXXFLAGS="${CXXFLAGS}" \
+		CPPFLAGS="${CPPFLAGS}" \
+		LDFLAGS="${LDFLAGS}" \
+		CXX="$(tc-getCXX)" \
+		AR="$(tc-getAR)" \
+		RANLIB="$(tc-getRANLIB)" \
+		SHARED=on \
+		NTL_GMP_LIP=on \
+		NTL_GF2X_LIB=on \
+		NTL_THREADS=$(usex threads on off) \
+		NTL_ENABLE_AVX_FFT=off \
+		NATIVE=on \
+		|| die "DoConfig failed"
+
+	if use doc; then
+		DOCS+=( "${WORKDIR}/${P}"/doc/*.txt )
+		HTML_DOCS=( "${WORKDIR}/${P}"/doc/*.html "${WORKDIR}/${P}"/doc/*.gif )
+	fi
+
+	# 780534 - Required for rlibtool so it can find the generated libtool
+	ln -sf libtool-build/libtool . || die
+}
+
+src_install() {
+	default
+	find "${ED}" -name '*.la' -delete || die
+
+	# Use rm -f because the static archive may not be created when
+	# using (for example) slibtool-shared.
+	rm -f "${ED}/usr/$(get_libdir)"/libntl.a || die
+
+	rm -r "${ED}"/usr/share/doc/NTL || die
+}


             reply	other threads:[~2023-06-14 11:15 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-14 11:15 Michael Orlitzky [this message]
  -- strict thread matches above, loose matches on Subject: below --
2023-11-24 16:26 [gentoo-commits] repo/gentoo:master commit in: dev-libs/ntl/ Michał Górny
2023-08-04 20:08 Michael Orlitzky
2023-08-04 10:59 Sam James
2023-07-15 22:01 Michael Orlitzky
2023-06-14 15:33 Arthur Zamarin
2023-06-14 15:15 Michael Orlitzky
2023-06-14 12:31 Arthur Zamarin
2023-06-07  4:21 Arthur Zamarin
2023-06-07  4:21 Arthur Zamarin
2023-06-07  4:21 Arthur Zamarin
2023-06-05 13:02 Michael Orlitzky
2023-06-05 13:02 Michael Orlitzky
2023-04-16 12:11 WANG Xuerui
2022-11-18 17:29 Arthur Zamarin
2022-06-20  2:03 Sam James
2022-05-07 12:20 Michael Orlitzky
2022-05-06 23:45 Jakov Smolić
2022-05-06 23:39 Jakov Smolić
2022-05-06 23:39 Jakov Smolić
2022-03-26  0:01 Sam James
2022-01-23 23:47 Michael Orlitzky
2022-01-18 14:49 Michael Orlitzky
2021-11-22  3:07 Yixun Lan
2021-11-09  3:37 Michael Orlitzky
2021-11-09  3:37 Michael Orlitzky
2021-11-03 12:50 Michael Orlitzky
2021-11-03 10:08 Agostino Sarubbo
2021-11-03 10:06 Agostino Sarubbo
2021-11-02 15:46 Sam James
2021-03-16 21:33 Sam James
2021-03-06  2:57 Michael Orlitzky
2021-01-06 19:34 Fabian Groffen
2020-08-25 10:25 Sam James
2020-04-23  6:29 Agostino Sarubbo
2020-04-23  6:24 Agostino Sarubbo
2020-04-23  6:20 Agostino Sarubbo
2020-03-24 18:22 Michael Orlitzky
2017-07-21  0:23 Sebastien Fabbro
2015-08-26 11:00 Agostino Sarubbo

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=1686741271.9ed8c182abf1232c57446017da57b24c4351a34c.mjo@gentoo \
    --to=mjo@gentoo.org \
    --cc=gentoo-commits@lists.gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /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