Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 358847 - Remove old-style virtual/portage
Summary: Remove old-style virtual/portage
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 350792
  Show dependency tree
 
Reported: 2011-03-14 11:50 UTC by Ulrich Müller
Modified: 2011-04-13 13:55 UTC (History)
7 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Proposed package-manager-0.ebuild (package-manager-0.ebuild,608 bytes, text/plain)
2011-04-08 19:54 UTC, Ulrich Müller
Details
quick fix for catalyst (catalyst.patch,472 bytes, patch)
2011-04-10 05:12 UTC, Zac Medico
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ulrich Müller gentoo-dev 2011-03-14 11:50:12 UTC
See tracker bug 350792.

virtual/portage
   packages:
      sys-apps/paludis
      sys-apps/portage

Please convert to a new-style virtual.
Comment 1 Ciaran McCreesh 2011-03-16 14:46:34 UTC
Perhaps we should use virtual/package-manager instead. AFAIK the only use for this virtual is for the system set.
Comment 2 Ulrich Müller gentoo-dev 2011-04-04 22:07:52 UTC
<ulm> zmedico: do you have an opinion on bug 358847 comment #1?
<willikins> https://bugs.gentoo.org/358847 "Remove old-style virtual/portage";
            Gentoo Linux, Eclasses and Profiles; NEW; ulm:dev-portage
<zmedico> ulm: as long as there are no reverse dependencies then there's no
          cost in renaming it to virtual/package-manager. I don't have any
          preference about what we call it.
<ulm> zmedico: indeed it has no reverse dependencies
<ulm> it appears only in profiles/base/packages and in packages.build of some
      subprofiles


@ferringb: Shouldn't pkgcore be another possible dependency of the virtual?
Comment 3 Ulrich Müller gentoo-dev 2011-04-08 19:54:14 UTC
Created attachment 269061 [details]
Proposed package-manager-0.ebuild

Proposed ebuild is attached, please review.

Would dev-portage@g.o be o.k. as maintainer in metadata? (I think there won't be much to maintain anyway.)
Comment 4 Zac Medico gentoo-dev 2011-04-08 22:51:02 UTC
(In reply to comment #3)
> Would dev-portage@g.o be o.k. as maintainer in metadata? (I think there won't
> be much to maintain anyway.)

That's fine, though base-system might be more appropriate since the virtual exists mainly for the purpose of defining the system set.
Comment 5 Ulrich Müller gentoo-dev 2011-04-09 21:30:10 UTC
virtual/package-manager committed to tree and profiles updated.
Please remove the PROVIDE="virtual/portage" line from your ebuilds.
Comment 6 Zac Medico gentoo-dev 2011-04-10 02:49:38 UTC
Okay, PROVIDE is no longer in sys-apps/portage ebuilds.
Comment 7 Zac Medico gentoo-dev 2011-04-10 05:04:45 UTC
It just realized that this change might break out auto stage builds since catalyst seems to have a hardcoded reference to virtual/portage.
Comment 8 Zac Medico gentoo-dev 2011-04-10 05:12:58 UTC
Created attachment 269249 [details, diff]
quick fix for catalyst

Currently, emerge doesn't have an a way to force the dependencies of the new-style virtual/package-manager ebuild to be re-merged. So, the closest and simplest fix is to hardcode sys-apps/portage into catalyst.
Comment 9 Raúl Porcel (RETIRED) gentoo-dev 2011-04-10 21:17:34 UTC
(In reply to comment #8)
> Created attachment 269249 [details, diff]
> quick fix for catalyst
> 
> Currently, emerge doesn't have an a way to force the dependencies of the
> new-style virtual/package-manager ebuild to be re-merged. So, the closest and
> simplest fix is to hardcode sys-apps/portage into catalyst.

Applied, on catalyst-2.0.6.915
Comment 10 Ulrich Müller gentoo-dev 2011-04-12 21:11:19 UTC
(In reply to comment #5)
> Please remove the PROVIDE="virtual/portage" line from your ebuilds.

@paludis team: Great that you not only don't remove the PROVIDE from existing ebuilds, but even add new ones (0.60.4) with it. :(
Comment 11 Ciaran McCreesh 2011-04-12 21:16:01 UTC
Don't think the Paludis team is Cc:ed...
Comment 12 Ulrich Müller gentoo-dev 2011-04-12 21:18:32 UTC
(In reply to comment #11)
> Don't think the Paludis team is Cc:ed...

Maintainer from metadata was, maybe that should be fixed then?
Comment 13 Robert Piasek (RETIRED) gentoo-dev 2011-04-13 12:39:33 UTC
It should be all fixed now in paludis ebuilds. I also added myself to metadata.
Comment 14 Ulrich Müller gentoo-dev 2011-04-13 13:55:04 UTC
Thanks everybody. Closing.