Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 358993 - Remove old-style virtual/imap-c-client
Summary: Remove old-style virtual/imap-c-client
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Net-Mail Packages
URL:
Whiteboard:
Keywords:
Depends on: 359221
Blocks: 350792
  Show dependency tree
 
Reported: 2011-03-15 10:54 UTC by Ulrich Müller
Modified: 2011-03-17 18:58 UTC (History)
1 user (show)

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


Attachments

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-15 10:54:54 UTC
See tracker bug 350792.

virtual/imap-c-client
   packages:
      net-libs/c-client
      net-mail/uw-imap

Please convert to a new-style virtual.

Note:
- Used in blockers, which however shouldn't be an issue with only two packages
  providing the virtual.
Comment 1 Eray Aslan gentoo-dev 2011-03-16 13:56:53 UTC
Done.  Closing.
Comment 2 Rafał Mużyło 2011-03-16 21:47:45 UTC
Given bug 359221, reopening.

Unsure whether this needs a fix in the virtual or in php-common-r1.eclass.
Unsure whether php herd isn't currently defunct.
Comment 3 Ole Markus With (RETIRED) gentoo-dev 2011-03-16 21:58:16 UTC
Unsure what you mean by defunct. It is certainly understaffed. But I do not see how this bug is caused by the php herd.

Anyways, as far as I can see, the functionality from php-common-r1.eclass's php_check_imap need to be moved into the ebuilds as USE dependencies.
Comment 4 Ulrich Müller gentoo-dev 2011-03-16 22:49:28 UTC
A possible solution would be to add IUSE="ssl" to the virtual, plus appropriate USE dependencies (like net-libs/c-client[ssl]).
Comment 5 Ole Markus With (RETIRED) gentoo-dev 2011-03-17 10:11:20 UTC
That would be a good solution. If you can take care of the virtual, I can take care of the php ebuilds/eclass.
Comment 6 Eray Aslan gentoo-dev 2011-03-17 11:05:21 UTC
+*imap-c-client-1 (17 Mar 2011)
+
+  17 Mar 2011; Eray Aslan <eras@gentoo.org> +imap-c-client-1.ebuild:
+  add ssl USE flag - bugs 359221 358993
+

Please reopen if you still have a problem.
Comment 7 Rafał Mużyło 2011-03-17 14:38:25 UTC
Well, perhaps saying "defunct" was a bit much, but last time I checked it was reduced to two people after that time lead resigned.
And there's i.e. bug 343699, which, while not that important to me,  is basically an ebuild *still* unported to slotted php.
Comment 8 Ole Markus With (RETIRED) gentoo-dev 2011-03-17 16:24:56 UTC
It almost works now. Could you change the deps to the following?

RDEPEND=" || (  net-libs/c-client[ssl=]
                net-mail/uw-imap[ssl=] )"


@Rafał Mużyło: It is actually only I who is still active in the herd now. That bug, however, is not assigned to php, nor that package's php its maintainer.
Comment 9 Rafał Mużyło 2011-03-17 18:27:05 UTC
OK, I missed that part, but on the move to slotted either package maintainer should have been pinged about the needed change, or it should be done alike to how gnome herd handles gtk3 ("fix it on your own or we'll do it for you" - while it might be annoying on occasions, it's still quite effective).
Comment 10 Eray Aslan gentoo-dev 2011-03-17 18:58:42 UTC
(In reply to comment #8)
> It almost works now. Could you change the deps to the following?

{ssl?] should have been good enough.  Why does php care?  Anyway, took the easy way out and changed to [ssl=].