public inbox for gentoo-pms@lists.gentoo.org
 help / color / mirror / Atom feed
From: Brian Harring <ferringb@gmail.com>
To: gentoo-pms@lists.gentoo.org
Subject: Re: [gentoo-pms] [PATCH] Remove cat/foo:* slot operator.
Date: Mon, 10 Sep 2012 00:07:27 -0700	[thread overview]
Message-ID: <20120910070727.GC8036@localhost> (raw)
In-Reply-To: <20120910073947.2a0209fe@googlemail.com>

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


      reply	other threads:[~2012-09-10  7:07 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120910070727.GC8036@localhost \
    --to=ferringb@gmail.com \
    --cc=gentoo-pms@lists.gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox