Re: [gentoo-dev] [PATCH] cargo.eclass: Emit a warning if the package uses 300+ crates

2025-01-14 Thread Michał Górny
On Tue, 2025-01-14 at 17:56 +0100, Florian Schmaus wrote: > > It certainly feels like all of a sudden to me. At least, as far as I > understand, there is no trigger event or similar. I am sorry, but > instead, it appears that you have decided that today is the day when we > need this. I know i

Re: [gentoo-dev] [PATCH] cargo.eclass: Emit a warning if the package uses 300+ crates

2025-01-14 Thread Florian Schmaus
On 13/01/2025 14.36, Michał Górny wrote: On Mon, 2025-01-13 at 10:40 +0100, Florian Schmaus wrote: First, switching from individual crates to a single crate tarball disallows inter-package crate archive reuse. Often, users will already have the required crates downloaded because another installe

Re: [gentoo-dev] [PATCH] cargo.eclass: Emit a warning if the package uses 300+ crates

2025-01-13 Thread Ionen Wolkens
On Mon, Jan 13, 2025 at 05:23:54AM -0800, orbea wrote: > On Mon, 13 Jan 2025 10:40:30 +0100 > Florian Schmaus wrote: > > > On 12/01/2025 13.56, Michał Górny wrote: > > > Emit a QA warning suggesting the use of crate tarball, when the > > > package in question uses 300 crates or more. Such a long

Re: [gentoo-dev] [PATCH] cargo.eclass: Emit a warning if the package uses 300+ crates

2025-01-13 Thread Michał Górny
On Mon, 2025-01-13 at 10:40 +0100, Florian Schmaus wrote: > First, switching from individual crates to a single crate tarball > disallows inter-package crate archive reuse. Often, users will already > have the required crates downloaded because another installed package > used them. With an arti

Re: [gentoo-dev] [PATCH] cargo.eclass: Emit a warning if the package uses 300+ crates

2025-01-13 Thread orbea
On Mon, 13 Jan 2025 10:40:30 +0100 Florian Schmaus wrote: > On 12/01/2025 13.56, Michał Górny wrote: > > Emit a QA warning suggesting the use of crate tarball, when the > > package in question uses 300 crates or more. Such a long crate > > lists cause ebuilds and Manifests to grow very fast, cau

Re: [gentoo-dev] [PATCH] cargo.eclass: Emit a warning if the package uses 300+ crates

2025-01-13 Thread Florian Schmaus
On 12/01/2025 13.56, Michał Górny wrote: Emit a QA warning suggesting the use of crate tarball, when the package in question uses 300 crates or more. Such a long crate lists cause ebuilds and Manifests to grow very fast, causing significant space consumption on end user systems (including users

Re: [gentoo-dev] [PATCH] cargo.eclass: Emit a warning if the package uses 300+ crates

2025-01-12 Thread Ionen Wolkens
On Sun, Jan 12, 2025 at 02:30:10PM +, Alexey Sokolov wrote: > 12.01.2025 13:15, Agostino Sarubbo пишет: > > > On domenica 12 gennaio 2025 13:56:39 CET Michał Górny wrote: > > > >> + if [[ ${#crates[@]} -ge 300 ]]; then > > > >> + eqawarn "This package uses a very large number of > > > >> CRATE

Re: [gentoo-dev] [PATCH] cargo.eclass: Emit a warning if the package uses 300+ crates

2025-01-12 Thread Alexey Sokolov
12.01.2025 13:15, Agostino Sarubbo пишет: > On domenica 12 gennaio 2025 13:56:39 CET Michał Górny wrote: > >> + if [[ ${#crates[@]} -ge 300 ]]; then > >> + eqawarn "This package uses a very large number of > >> CRATES. Please provide" + eqawarn "a crate tarball > >> instead and fetch it via SRC_UR

Re: [gentoo-dev] [PATCH] cargo.eclass: Emit a warning if the package uses 300+ crates

2025-01-12 Thread Agostino Sarubbo
On domenica 12 gennaio 2025 13:56:39 CET Michał Górny wrote: > + if [[ ${#crates[@]} -ge 300 ]]; then > + eqawarn "This package uses a very large number of > CRATES. Please provide" + eqawarn "a crate tarball > instead and fetch it via SRC_

[gentoo-dev] [PATCH] cargo.eclass: Emit a warning if the package uses 300+ crates

2025-01-12 Thread Michał Górny
Emit a QA warning suggesting the use of crate tarball, when the package in question uses 300 crates or more. Such a long crate lists cause ebuilds and Manifests to grow very fast, causing significant space consumption on end user systems (including users who are not using the package in question)