Bug 285566

Summary:	mail-notification should not depend on eel
Product:	Gentoo Linux	Reporter:	Francisco J. Vazquez <dv>
Component:	[OLD] GNOME	Assignee:	Hans de Graaff <graaff>
Status:	RESOLVED FIXED
Severity:	normal	CC:	net-mail+disabled, zgredder
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---

Description Francisco J. Vazquez 2009-09-19 13:02:45 UTC

Since eel is no longer used by gnome and it was only pulled by mail-notification, I uninstalled it and removed it from RDEPEND. Mail-notification still compiles and runs fine. In fact the NEWS file says:

* libeel is no longer used since its API is not stable (#180400)

... that's from march 2008.


I'll continue to use my custom ebuild (I don't want or need gmime) but I thought someone would like having one less deprecated dependency :)

Reproducible: Always

Comment 1 Bartosz Szreder 2009-09-20 01:03:25 UTC

How about posting a diff between your custom ebuild and official ebuild? It would help devs getting it fixed in the tree.

Comment 2 Francisco J. Vazquez 2009-09-20 07:40:29 UTC

(In reply to comment #1)
> How about posting a diff between your custom ebuild and official ebuild? It
> would help devs getting it fixed in the tree.
> 

I usually post a patch when the differences are complex. In this case it's just removing the "eel" line from RDEPEND, as I said. It doesn't make much sense to provide a patch for that.

Comment 3 Bartosz Szreder 2009-09-20 10:45:50 UTC

Post a patch anyway, even if it's trivial.

Comment 4 Francisco J. Vazquez 2009-09-20 12:55:11 UTC

(In reply to comment #3)
> Post a patch anyway, even if it's trivial.
> 

We're talking about removing a frikkin line in a short file. No, I won't provide a patch for that. If the bug takes longer to fix or is marked as wontfix because of that, fine by me. I already said that I'd use my ebuild anyway.

Comment 5 Hans de Graaff gentoo-dev

2009-09-20 13:55:19 UTC

Agreed, eel is no longer used since version 5.1. I missed that somehow at the time. I've removed the dependency in CVS. Thanks.

BTW, feel free to open a new bug for the gmime stuff. Looking at my comment it didn't seem to make sense to make it optional at the time, but I'd be happy to revisit that decision.

Comment 6 Francisco J. Vazquez 2009-09-20 17:00:35 UTC

(In reply to comment #5)
> Agreed, eel is no longer used since version 5.1. I missed that somehow at the
> time. I've removed the dependency in CVS. Thanks.
> 
> BTW, feel free to open a new bug for the gmime stuff. Looking at my comment it
> didn't seem to make sense to make it optional at the time, but I'd be happy to
> revisit that decision.
> 

No, I think you were right when you decided to force it; *all* the configure options except gmail and evo depend on gmime, so you'd need to add a dozen "use? ( gmime )" lines. I just need gmail, so I simply removed the dep, but I think the ebuild in portage is fine with the forced dependency.