Re: [gentoo-dev] [PATCH 1/7] eclass/nuget.eclass: introduce new eclass

2023-07-31 Thread Florian Schmaus
On 31/07/2023 15.53, Michał Górny wrote: On Mon, 2023-07-31 at 12:49 +0200, Florian Schmaus wrote: On 31/07/2023 11.32, Sam James wrote: Florian Schmaus writes: [[PGP Signed Part:Undecided]] On 31/07/2023 07.02, Michał Górny wrote: On Sun, 2023-07-30 at 22:19 +0200, Florian Schmaus wrote:

Re: [gentoo-dev] [PATCH 1/7] eclass/nuget.eclass: introduce new eclass

2023-07-31 Thread Michał Górny
On Mon, 2023-07-31 at 12:49 +0200, Florian Schmaus wrote: > On 31/07/2023 11.32, Sam James wrote: > > > > Florian Schmaus writes: > > > > > [[PGP Signed Part:Undecided]] > > > On 31/07/2023 07.02, Michał Górny wrote: > > > > On Sun, 2023-07-30 at 22:19 +0200, Florian Schmaus wrote: > > > > >

Re: [gentoo-dev] [PATCH 1/7] eclass/nuget.eclass: introduce new eclass

2023-07-31 Thread Sam James
Florian Schmaus writes: > [[PGP Signed Part:Undecided]] > On 31/07/2023 11.32, Sam James wrote: >> Florian Schmaus writes: >> >>> [[PGP Signed Part:Undecided]] >>> On 31/07/2023 07.02, Michał Górny wrote: On Sun, 2023-07-30 at 22:19 +0200, Florian Schmaus wrote: > Which problem are

Re: [gentoo-dev] [PATCH 1/7] eclass/nuget.eclass: introduce new eclass

2023-07-31 Thread Florian Schmaus
On 31/07/2023 11.32, Sam James wrote: Florian Schmaus writes: [[PGP Signed Part:Undecided]] On 31/07/2023 07.02, Michał Górny wrote: On Sun, 2023-07-30 at 22:19 +0200, Florian Schmaus wrote: Which problem are we solving by moving away from this towards a slightly more verbose construct?

Re: [gentoo-dev] [PATCH 1/7] eclass/nuget.eclass: introduce new eclass

2023-07-31 Thread Sam James
Florian Schmaus writes: > [[PGP Signed Part:Undecided]] > On 31/07/2023 07.02, Michał Górny wrote: >> On Sun, 2023-07-30 at 22:19 +0200, Florian Schmaus wrote: >>> Which problem are we solving by moving away from this towards a slightly >>> more verbose construct? >> The problem was that

Re: [gentoo-dev] [PATCH 1/7] eclass/nuget.eclass: introduce new eclass

2023-07-31 Thread Florian Schmaus
On 31/07/2023 07.02, Michał Górny wrote: On Sun, 2023-07-30 at 22:19 +0200, Florian Schmaus wrote: Which problem are we solving by moving away from this towards a slightly more verbose construct? The problem was that cargo.eclass ebuilds were taking significant time during cache regeneration

Re: [gentoo-dev] [PATCH 1/7] eclass/nuget.eclass: introduce new eclass

2023-07-30 Thread Michał Górny
On Sun, 2023-07-30 at 22:01 +0200, Maciej Barć wrote: > > > +# @ECLASS_VARIABLE: NUGETS > > > +# @DEFAULT_UNSET > > > +# @PRE_INHERIT > > > +# @DESCRIPTION: > > > +# String containing all NuGet packages that need to be downloaded. > > > +# Used by "nuget_uris". > > > +# > > > +# Example: > > > +#

Re: [gentoo-dev] [PATCH 1/7] eclass/nuget.eclass: introduce new eclass

2023-07-30 Thread Michał Górny
On Sun, 2023-07-30 at 22:19 +0200, Florian Schmaus wrote: > Which problem are we solving by moving away from this towards a slightly > more verbose construct? The problem was that cargo.eclass ebuilds were taking significant time during cache regeneration and slowing down tools noticeably. No

Re: [gentoo-dev] [PATCH 1/7] eclass/nuget.eclass: introduce new eclass

2023-07-30 Thread Florian Schmaus
On 30/07/2023 21.30, Michał Górny wrote: On Sun, 2023-07-30 at 16:26 +0200, Maciej Barć wrote: +# @FUNCTION: nuget_uris +# @USAGE: +# @DESCRIPTION: +# Generates the URIs to put in SRC_URI to help fetch dependencies. +# If no arguments provided, uses the "NUGETS" variable. Sounds like you're

Re: [gentoo-dev] [PATCH 1/7] eclass/nuget.eclass: introduce new eclass

2023-07-16 Thread Maciej Barć
Offtopic: That calls for a YAS snippet ;D W dniu 16.07.2023 o 15:40, Ulrich Mueller pisze: On Sun, 16 Jul 2023, Maciej Barć wrote: +case "${EAPI}" in + 7 | 8 ) + : + ;; + * ) + die "${ECLASS}: EAPI ${EAPI} unsupported." + ;;

Re: [gentoo-dev] [PATCH 1/7] eclass/nuget.eclass: introduce new eclass

2023-07-16 Thread Ulrich Mueller
> On Sun, 16 Jul 2023, Maciej Barć wrote: > +case "${EAPI}" in > + 7 | 8 ) > + : > + ;; > + * ) > + die "${ECLASS}: EAPI ${EAPI} unsupported." > + ;; > +esac The QA team has invested quite some work to unify that case block between