Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 196363

Summary: You don't actually need firefox or seamonkey to build galeon with xulrunner
Product: Gentoo Linux Reporter: Andrew Cowie <andrew>
Component: New packagesAssignee: Gentoo Linux bug wranglers <bug-wranglers>
Status: RESOLVED NEEDINFO    
Severity: minor    
Priority: High    
Version: 2007.0   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: Make back end dependency orthogonal

Description Andrew Cowie 2007-10-19 07:28:12 UTC
Many months back I switched to using xulrunner only (mostly on the advice of the GNOME hackers at GUADEC).

When I tried to do this for Galeon, I found that I had to manually edit a copy of the .ebuild to permit it to build USE="xulrunner -seamonkey -firefox". My diff is attached.

AfC
Comment 1 Andrew Cowie 2007-10-19 07:29:51 UTC
Created attachment 133865 [details, diff]
Make back end dependency orthogonal

Simple change making the seamonkey, firefox, and xulrunner USE flags orthogonal. Don't really expect anyone to apply this as-is, but I thought I'd point out the general idea of what I was up to.

AfC
Comment 2 Jakub Moc (RETIRED) gentoo-dev 2007-10-19 07:33:41 UTC
I completely fail to see what are you fixing here, sorry. The patch is broken and pulls in dependencies that will be completely unused when seamonkey/firefox use flags are unused.
Comment 3 Andrew Cowie 2007-10-19 09:10:52 UTC
I ran into the problem that without this patch, emerging USE=-firefox xulrunner still resulted in the firefox backend being used.

AfC
Comment 4 Jakub Moc (RETIRED) gentoo-dev 2007-10-19 09:16:56 UTC
(In reply to comment #3)
> I ran into the problem that without this patch, emerging USE=-firefox xulrunner
> still resulted in the firefox backend being used.

There's no firefox use flag in the ebuild, so... If you claim there's some bug, tell us how to reproduce it. The patch is wrong as said above, and  myconf="--with-mozilla=xulrunner" w/ USE=xulrunnner is completely unchanged by your patch either.