Bug 358847 – Remove old-style virtual/portage

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description Ulrich Müller 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 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 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 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 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 2011-04-10 02:49:38 UTC]

Okay, PROVIDE is no longer in sys-apps/portage ebuilds.

[Comment 7 Zac Medico 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 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) 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 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 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) 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 2011-04-13 13:55:04 UTC]

Thanks everybody. Closing.