* [gentoo-pms] [PATCH] Remove cat/foo:* slot operator. @ 2012-09-09 20:12 Brian Harring 2012-09-10 6:25 ` Ulrich Mueller 2012-09-10 6:39 ` Ciaran McCreesh 0 siblings, 2 replies; 4+ messages in thread From: Brian Harring @ 2012-09-09 20:12 UTC (permalink / raw To: gentoo-pms; +Cc: Brian Harring Introduced originally via 8b54be6a, this is effectively a noop for gentoo ebuilds- nor is it providing any useful information to either the dev, or the PM, thus remove it. --- dependencies.tex | 3 --- 1 file changed, 3 deletions(-) diff --git a/dependencies.tex b/dependencies.tex index 47c0261..96d22ac 100644 --- a/dependencies.tex +++ b/dependencies.tex @@ -288,9 +288,6 @@ delimited by a \t{/} character. the following operators: \begin{description} -\item[*] Indicates that any slot value is acceptable. In addition, for runtime dependencies, -indicates that the package will not break if the matched package is uninstalled and replaced by -a different matching package in a different slot. \item[=] Indicates that any slot value is acceptable. In addition, for runtime dependencies, indicates that the package will break unless a matching package with slot and sub-slot equal to the slot and sub-slot of the best installed version at the time the package was installed is available. -- 1.7.12 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [gentoo-pms] [PATCH] Remove cat/foo:* slot operator. 2012-09-09 20:12 [gentoo-pms] [PATCH] Remove cat/foo:* slot operator Brian Harring @ 2012-09-10 6:25 ` Ulrich Mueller 2012-09-10 6:39 ` Ciaran McCreesh 1 sibling, 0 replies; 4+ messages in thread From: Ulrich Mueller @ 2012-09-10 6:25 UTC (permalink / raw To: gentoo-pms; +Cc: Brian Harring >>>>> On Sun, 9 Sep 2012, Brian Harring wrote: > Introduced originally via 8b54be6a, this is effectively a noop for > gentoo ebuilds- nor is it providing any useful information to either > the dev, or the PM, thus remove it. The :* operator had been approved by a previous council for EAPI 4 (which was still called EAPI 3 at the time): http://www.gentoo.org/proj/en/council/meeting-logs/20090514-summary.txt Discussion is here: http://www.gentoo.org/proj/en/council/meeting-logs/20090423.txt Ulrich ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [gentoo-pms] [PATCH] Remove cat/foo:* slot operator. 2012-09-09 20:12 [gentoo-pms] [PATCH] Remove cat/foo:* slot operator Brian Harring 2012-09-10 6:25 ` Ulrich Mueller @ 2012-09-10 6:39 ` Ciaran McCreesh 2012-09-10 7:07 ` Brian Harring 1 sibling, 1 reply; 4+ messages in thread From: Ciaran McCreesh @ 2012-09-10 6:39 UTC (permalink / raw To: gentoo-pms [-- Attachment #1: Type: text/plain, Size: 708 bytes --] On Sun, 9 Sep 2012 13:12:43 -0700 Brian Harring <ferringb@gmail.com> wrote: > Introduced originally via 8b54be6a, this is effectively a noop for > gentoo ebuilds- nor is it providing any useful information to either > the dev, or the PM, thus remove it. The point of :* is to allow repoman to detect situations where a package has a build and a run dep upon something that has multiple slots, and to remind the developer that they either need to := it, or to explicitly say that it's ok for slots to change. Without :*, there's no way to tell the difference between "the developer forgot, or slots were added in later", and "this really can switch slots at runtime". -- Ciaran McCreesh [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 198 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [gentoo-pms] [PATCH] Remove cat/foo:* slot operator. 2012-09-10 6:39 ` Ciaran McCreesh @ 2012-09-10 7:07 ` Brian Harring 0 siblings, 0 replies; 4+ messages in thread From: Brian Harring @ 2012-09-10 7:07 UTC (permalink / raw To: gentoo-pms On Mon, Sep 10, 2012 at 07:39:47AM +0100, Ciaran McCreesh wrote: > On Sun, 9 Sep 2012 13:12:43 -0700 > Brian Harring <ferringb@gmail.com> wrote: > > Introduced originally via 8b54be6a, this is effectively a noop for > > gentoo ebuilds- nor is it providing any useful information to either > > the dev, or the PM, thus remove it. > > The point of :* is to allow repoman to detect situations where a > package has a build and a run dep upon something that has multiple > slots, and to remind the developer that they either need to := it, or > to explicitly say that it's ok for slots to change. Related, slot dep operators don't have any meaning in DEPEND; shouldn't be usable in that case then (considering the binding behaviour, it's only relevant to PDEPEND/RDEPEND). > Without :*, there's no way to tell the difference between "the > developer forgot, or slots were added in later", and "this really can > switch slots at runtime". The implication of this is that basically every damn ebuild's going to grow :* however; that's rather ugly, and I'm not particularly sure that's what folks signed up for. Either way, the wording should include this justification; as is, it reads as a no-op. Current wording: """ Indicates that any slot value is acceptable. In addition, for runtime dependencies, indicates that the package will not break if the matched package is uninstalled and replaced by a different matching package in a different slot. """ alternative- """ Indicate that despite multiple slottings matching, this is known, and acceptable for the slotting to change at runtime. Ebuilds by default should not use unless necessary; the absense of this operator can be used by QA tools to detect potential breakage when a pkg is being converted to slotted. """ Roughly. English ain't the forte- point is, include the reasoning and the clarification for when it should actually be used (the original text describes what it does, not why/when you should use it). ~harring ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-09-10 7:07 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-09-09 20:12 [gentoo-pms] [PATCH] Remove cat/foo:* slot operator Brian Harring 2012-09-10 6:25 ` Ulrich Mueller 2012-09-10 6:39 ` Ciaran McCreesh 2012-09-10 7:07 ` Brian Harring
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox