* [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