public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
* [gentoo-dev] [PATCH] distutils-r1.eclass: Make use of gpep517's new build-wheel --prefix arg
@ 2023-09-27 21:32 James Le Cuirot
  2023-09-28  1:15 ` Sam James
  0 siblings, 1 reply; 3+ messages in thread
From: James Le Cuirot @ 2023-09-27 21:32 UTC (permalink / raw)
  To: gentoo-dev; +Cc: James Le Cuirot

This fixes cross-prefix installations.

Signed-off-by: James Le Cuirot <chewi@gentoo.org>
---
 eclass/distutils-r1.eclass | 1 +
 1 file changed, 1 insertion(+)

diff --git a/eclass/distutils-r1.eclass b/eclass/distutils-r1.eclass
index 56afcdc5bcb8..838162c6b0f8 100644
--- a/eclass/distutils-r1.eclass
+++ b/eclass/distutils-r1.eclass
@@ -1444,6 +1444,7 @@ distutils_pep517_install() {
 	einfo "  Building the wheel for ${PWD#${WORKDIR}/} via ${build_backend}"
 	local cmd=(
 		gpep517 build-wheel
+			--prefix="${EPREFIX}/usr"
 			--backend "${build_backend}"
 			--output-fd 3
 			--wheel-dir "${WHEEL_BUILD_DIR}"
-- 
2.41.0



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [gentoo-dev] [PATCH] distutils-r1.eclass: Make use of gpep517's new build-wheel --prefix arg
  2023-09-27 21:32 [gentoo-dev] [PATCH] distutils-r1.eclass: Make use of gpep517's new build-wheel --prefix arg James Le Cuirot
@ 2023-09-28  1:15 ` Sam James
  2023-09-28 21:33   ` James Le Cuirot
  0 siblings, 1 reply; 3+ messages in thread
From: Sam James @ 2023-09-28  1:15 UTC (permalink / raw)
  To: gentoo-dev; +Cc: James Le Cuirot


James Le Cuirot <chewi@gentoo.org> writes:

> This fixes cross-prefix installations.

Do we need to crank the minimum version we depend on for gpep517 then?

>
> Signed-off-by: James Le Cuirot <chewi@gentoo.org>
> ---
>  eclass/distutils-r1.eclass | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/eclass/distutils-r1.eclass b/eclass/distutils-r1.eclass
> index 56afcdc5bcb8..838162c6b0f8 100644
> --- a/eclass/distutils-r1.eclass
> +++ b/eclass/distutils-r1.eclass
> @@ -1444,6 +1444,7 @@ distutils_pep517_install() {
>  	einfo "  Building the wheel for ${PWD#${WORKDIR}/} via ${build_backend}"
>  	local cmd=(
>  		gpep517 build-wheel
> +			--prefix="${EPREFIX}/usr"
>  			--backend "${build_backend}"
>  			--output-fd 3
>  			--wheel-dir "${WHEEL_BUILD_DIR}"



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [gentoo-dev] [PATCH] distutils-r1.eclass: Make use of gpep517's new build-wheel --prefix arg
  2023-09-28  1:15 ` Sam James
@ 2023-09-28 21:33   ` James Le Cuirot
  0 siblings, 0 replies; 3+ messages in thread
From: James Le Cuirot @ 2023-09-28 21:33 UTC (permalink / raw)
  To: gentoo-dev

[-- Attachment #1: Type: text/plain, Size: 298 bytes --]

On Thu, 2023-09-28 at 02:15 +0100, Sam James wrote:
> James Le Cuirot <chewi@gentoo.org> writes:
> 
> > This fixes cross-prefix installations.
> 
> Do we need to crank the minimum version we depend on for gpep517 then?

Woops, yes. Got over-excited when I saw it was stabilised. Thanks. :)

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 858 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-09-28 21:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-27 21:32 [gentoo-dev] [PATCH] distutils-r1.eclass: Make use of gpep517's new build-wheel --prefix arg James Le Cuirot
2023-09-28  1:15 ` Sam James
2023-09-28 21:33   ` James Le Cuirot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox