[gentoo-dev] [PATCH 8/8] distutils-r1.eclass: Support cross-compiling with PyO3

2024-07-25 Thread James Le Cuirot
Only one variable needs to be set. For details, see https://pyo3.rs/latest/building-and-distribution.html#cross-compiling. Signed-off-by: James Le Cuirot --- eclass/distutils-r1.eclass | 3 +++ 1 file changed, 3 insertions(+) diff --git a/eclass/distutils-r1.eclass b/eclass/distutils-r1.eclass

[gentoo-dev] [PATCH 7/8] distutils-r1.eclass: Add python_get_stdlib helper function

2024-07-25 Thread James Le Cuirot
This is just like python_get_sitedir, but it returns the stdlib directory such as /usr/lib/python3.12. This is useful for locating the sysconfigdata file. Signed-off-by: James Le Cuirot --- eclass/python-utils-r1.eclass | 23 +++ 1 file changed, 23 insertions(+) diff --git a

[gentoo-dev] [PATCH 6/6] distutils-r1.eclass: Use cargo_env when appropriate for flag handling

2024-07-25 Thread James Le Cuirot
cargo_env handles linker flags and enables cross-compiling. It also handles LTO filtering, so we can remove that from this eclass. Signed-off-by: James Le Cuirot --- eclass/distutils-r1.eclass | 16 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/eclass/distutils-

[gentoo-dev] [PATCH 5/6] cargo.eclass: Shadow flag variables so that LTO filtering remains local

2024-07-25 Thread James Le Cuirot
This is currently done for Cargo by distutils-r1.eclass. The next commit will remove that code, leaving cargo_env responsible for it. Signed-off-by: James Le Cuirot --- eclass/cargo.eclass | 9 + 1 file changed, 9 insertions(+) diff --git a/eclass/cargo.eclass b/eclass/cargo.eclass inde

[gentoo-dev] [PATCH 4/6] cargo.eclass: Explicitly tell rustc not to strip binaries

2024-07-25 Thread James Le Cuirot
Most projects don't strip binaries in release mode by default, but there are exceptions like app-misc/broot. Signed-off-by: James Le Cuirot --- eclass/cargo.eclass | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/eclass/cargo.eclass b/eclass/cargo.eclass index 6ea37ec5e4e4

[gentoo-dev] [PATCH 3/6] cargo.eclass: Handle LDFLAGS and RUSTFLAGS better

2024-07-25 Thread James Le Cuirot
LDFLAGS are not currently honoured by Cargo builds at all. It would be particularly advantageous to honour -fuse-ld because alternative linkers like mold are known to be significantly faster at handling Rust. As things stand, the eclass sets the linker to CC when cross-compiling, but it does so er

[gentoo-dev] [PATCH 2/6] cargo.eclass: Add cargo_env helper and use it in compile, test, install

2024-07-25 Thread James Le Cuirot
Rust packages have a tendency to rebuild parts during test and install. It is not clear whether this can be addressed. We were therefore relying on some environment variables set during the compile phase for cross-compiling to work in the later phases. This is not ideal, especially if you need to b

[gentoo-dev] [PATCH 1/6] cargo.eclass: Use newer Cargo config file name

2024-07-25 Thread James Le Cuirot
"config" is deprecated and "config.toml" has been valid for ages. Signed-off-by: James Le Cuirot --- eclass/cargo.eclass | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/eclass/cargo.eclass b/eclass/cargo.eclass index 7db34efb4e174..aab28dbbac167 100644 --- a/eclass/cargo.

[gentoo-dev] [PATCH v4 0/6] cargo.eclass: Handle LDFLAGS and RUSTFLAGS better

2024-07-25 Thread James Le Cuirot
Sorry for yet another iteration of this. I noticed that the build host flags were not being applied to rustc when cross-compiling. Upon investigation, I found that this isn't actually possible right now. I have reworked it to be close to what we need when it does become possible. James Le Cuirot (

Re: [gentoo-dev] [PATCH 4/5 v3] cargo.eclass: Shadow flag variables so that LTO filtering remains local

2024-07-25 Thread Sam James
James Le Cuirot writes: > Signed-off-by: James Le Cuirot > --- > eclass/cargo.eclass | 9 + > 1 file changed, 9 insertions(+) > > diff --git a/eclass/cargo.eclass b/eclass/cargo.eclass > index 44d3f7ee31f59..9f0bffee0e048 100644 > --- a/eclass/cargo.eclass > +++ b/eclass/cargo.eclass >

Re: [gentoo-dev] [PATCH 1/5 v3] cargo.eclass: Add cargo_env helper and use it in compile, test, install

2024-07-25 Thread Sam James
James Le Cuirot writes: > Rust packages have a tendency to rebuild parts during test and install. > It is not clear whether this can be addressed. We were therefore relying > on some environment variables set during the compile phase for > cross-compiling to work in the later phases. This is not

[gentoo-dev] [PATCH 5/5 v3] distutils-r1.eclass: Use cargo_env when appropriate for flag handling

2024-07-25 Thread James Le Cuirot
cargo_env handles linker flags and enables cross-compiling. It also handles LTO filtering, so we can remove that from this eclass. Signed-off-by: James Le Cuirot --- eclass/distutils-r1.eclass | 16 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/eclass/distutils-

[gentoo-dev] [PATCH 4/5 v3] cargo.eclass: Shadow flag variables so that LTO filtering remains local

2024-07-25 Thread James Le Cuirot
Signed-off-by: James Le Cuirot --- eclass/cargo.eclass | 9 + 1 file changed, 9 insertions(+) diff --git a/eclass/cargo.eclass b/eclass/cargo.eclass index 44d3f7ee31f59..9f0bffee0e048 100644 --- a/eclass/cargo.eclass +++ b/eclass/cargo.eclass @@ -532,7 +532,16 @@ cargo_src_configure() {

[gentoo-dev] [PATCH 3/5 v3] cargo.eclass: Explicitly tell rustc not to strip binaries

2024-07-25 Thread James Le Cuirot
Most projects don't strip binaries in release mode by default, but there are exceptions like app-misc/broot. Signed-off-by: James Le Cuirot --- eclass/cargo.eclass | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/eclass/cargo.eclass b/eclass/cargo.eclass index 596598ca8585

[gentoo-dev] [PATCH 2/5 v3] cargo.eclass: Handle LDFLAGS and RUSTFLAGS better

2024-07-25 Thread James Le Cuirot
LDFLAGS are not currently honoured by Cargo builds at all. It would be particularly advantageous to honour -fuse-ld because alternative linkers like mold are known to be significantly faster at handling Rust. As things stand, the eclass sets the linker to CC when cross-compiling, but it does so er

[gentoo-dev] [PATCH 1/5 v3] cargo.eclass: Add cargo_env helper and use it in compile, test, install

2024-07-25 Thread James Le Cuirot
Rust packages have a tendency to rebuild parts during test and install. It is not clear whether this can be addressed. We were therefore relying on some environment variables set during the compile phase for cross-compiling to work in the later phases. This is not ideal, especially if you need to b

[gentoo-dev] [PATCH] java-utils-2.eclass: fix ejunit_(), java-pkg_getjars --build-only

2024-07-25 Thread Volkmar W. Pogatzki
Closes: https://bugs.gentoo.org/936557 Signed-off-by: Volkmar W. Pogatzki --- eclass/java-utils-2.eclass | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eclass/java-utils-2.eclass b/eclass/java-utils-2.eclass index 91f8fba8790b..8a062f2ad87e 100644 --- a/eclass/java-utils-2.ec

Re: [gentoo-dev] [PATCH] php-ext-source-r3.eclass: Rebuild exts should dev-lang/php[threads,debug] change.

2024-07-25 Thread Jaco Kroon
Based on no feedback I proceeded to push a PR available here: https://github.com/gentoo/gentoo/pull/37712 On 2024/07/23 12:42, Jaco Kroon wrote: If these use flags change then the extension dir changes too, requiring extensions to be rebuilt. The downside of this change is that different vers

[gentoo-dev] Additional Maintainers wanted: OpenLDAP et al

2024-07-25 Thread Robin H. Johnson
Hi, I'd like to put a call out for additional maintainers for the LDAP herd: Three packages in the herd: dev-db/lmdb net-nds/openldap sys-auth/nss_ldap Plus other LDAP packages that are in maintainer-needed: acct-group/ldap acct-user/ldap net-nds/ldapvi sys-auth/pam_ldap sys-fs/ldapfuse There a

Re: [gentoo-dev] [PATCH 1/3] cargo.eclass: Add cargo_env helper and use it in compile, test, install

2024-07-25 Thread James Le Cuirot
On Wed, 2024-07-24 at 18:14 -0400, Eli Schwartz wrote: > On 7/24/24 6:07 PM, James Le Cuirot wrote: > > Rust packages have a tendency to rebuild parts during test and install. > > It is not clear whether this can be addressed. We were therefore relying > > on some environment variables set during t

Re: [gentoo-dev] [PATCH] savedconfig.eclass: Drop support for EAPI 6

2024-07-25 Thread James Le Cuirot
On Thu, 2024-07-25 at 07:35 +0200, Ulrich Müller wrote: > Signed-off-by: Ulrich Müller > --- > eclass/savedconfig.eclass | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/eclass/savedconfig.eclass b/eclass/savedconfig.eclass > index cc5748543078..65a1f3bfd800