| 1 |
On Wednesday 25 January 2006 18:10, Donnie Berkholz wrote: |
| 2 |
> Jason Stubbs wrote: |
| 3 |
> > DEPEND="x11-base/xorg-x11" # wrong |
| 4 |
> > DEPEND="virtual/x11" # wrong |
| 5 |
> > DEPEND="|| ( x11? ( virtual/x11 ) )" # wrong |
| 6 |
> > DEPEND="|| ( misc/atoms virtual/x11 )" # right |
| 7 |
> > |
| 8 |
> > There's a small possibility that broken packages will be missed by this, |
| 9 |
> > but is there any chance that valid packages will be incorrectly flagged? |
| 10 |
> > If this gets a go-ahead, it'll be easy enough to get in for the next |
| 11 |
> > release (which is likely this coming Saturday). |
| 12 |
> |
| 13 |
> It sounds right. There should be no valid instance of virtual/x11 that |
| 14 |
> is not within an || dep. |
| 15 |
|
| 16 |
I've implemented and tested the check locally but haven't committed it yet. |
| 17 |
Repoman isn't really structured to allow for tests against a set of ebuilds |
| 18 |
so the checks are done on every version. There is also definitely one false |
| 19 |
positive (virtual/x11-6.8) so, for this and the fact that every version is |
| 20 |
tested, it would probably better to just make it a warning. Thoughts? |
| 21 |
|
| 22 |
-- |
| 23 |
Jason Stubbs |
| 24 |
-- |
| 25 |
gentoo-dev@g.o mailing list |
Archives