Bug 472759 - OLPC requirements for xulrunner in Fedora Mainline
OLPC requirements for xulrunner in Fedora Mainline
Product: Fedora
Classification: Fedora
Component: xulrunner (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Christopher Aillon
Fedora Extras Quality Assurance
: Reopened
Depends On:
Blocks: FedoraOLPCDelta
  Show dependency treegraph
Reported: 2008-11-24 07:48 EST by Peter Robinson
Modified: 2009-08-03 08:16 EDT (History)
6 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2009-08-03 08:16:12 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
dennis: fedora‑cvs-

Attachments (Terms of Use)

External Trackers
Tracker ID Priority Status Summary Last Updated
Mozilla Foundation 466870 None None None Never

  None (edit)
Description Peter Robinson 2008-11-24 07:48:47 EST
Please create OLPC-4 branch for xulrunner
Comment 1 Dennis Gilmore 2008-11-25 11:26:07 EST
why?  there really is no reason for it. xulrunner in F-10 has  pyxpcom built. OLPC can and should use the Fedora branch.
Comment 2 Peter Robinson 2008-11-25 11:35:49 EST
If you look at the OLPC-3 branch there is also a number of patches to XULrunner to do with disable of default themes, and removing libgnome and gnomevfs. 

I'm sure that one day they will follow the gnome upstream and migrate to gio and all the new gnome stuff but its not in for 3.0 as far as I can see and I'm not sure its even there for 3.1. I'm investigating too see if the other patches are upstreamable and if the developer has filed patches in b.m.o for them but I'm not there yet. Believe me I want upstream OLPC everything ASAP :)

See the diff.

[perobinson@neo xulrunner]$ diff -u F-10/xulrunner.spec OLPC-3/xulrunner.spec 
--- F-10/xulrunner.spec	2008-11-12 22:37:53.000000000 +0000
+++ OLPC-3/xulrunner.spec	2008-10-07 14:48:11.000000000 +0100
@@ -7,8 +7,8 @@
 Summary:        XUL Runtime for Gecko Applications
 Name:           xulrunner
-Release:        1%{?dist}
+Release:        2%{?dist}
 URL:            http://developer.mozilla.org/En/XULRunner
 License:        MPLv1.1 or GPLv2+ or LGPLv2+
 Group:          Applications/Internet
@@ -32,6 +32,11 @@
 Patch26:        mozilla-ps-pdf-simplify-operators.patch
 Patch27:        mozilla-ssl-exception.patch
+Patch201:       xulrunner-olpc-no-native-theme.patch
+Patch202:       xulrunner-olpc-pre-dpi.patch
+Patch204:       xulrunner-olpc-xds.patch
+Patch205:	xulrunner-olpc-perms.patch
 # ---------------------------------------------------
@@ -47,9 +52,7 @@
 BuildRequires:  lcms-devel >= 1.17
 BuildRequires:  libIDL-devel
 BuildRequires:  gtk2-devel
-BuildRequires:  gnome-vfs2-devel
-BuildRequires:  libgnome-devel
-BuildRequires:  libgnomeui-devel
+BuildRequires:  dbus-glib-devel
 BuildRequires:  krb5-devel
 BuildRequires:  pango-devel
 BuildRequires:  freetype-devel >= 2.1.9
@@ -128,7 +131,12 @@
 %patch10 -p1 -b .pk
 %patch26 -p1 -b .ps-pdf-simplify-operators
+#%patch27 -p1 -b .ssl-exception
+%patch201 -p0 -b .no-native-theme
+%patch202 -p0 -b .dpi
+%patch204 -p0 -b .xds
+%patch205 -p0 -b .perms
 %{__rm} -f .mozconfig
 %{__cp} %{SOURCE10} .mozconfig
Comment 3 Dennis Gilmore 2008-11-25 13:11:06 EST
Please send the patches upstream and to Chris with explanations as to what they do and why they are needed.
Comment 4 Christopher Aillon 2008-11-25 16:40:09 EST
And at the bare minimum, I strongly suggest getting explicit approval from upstream (Mozilla) for these patches.  We have a great relationship with Mozilla which is good when we need to seek changes for things such as the EULA issue.  Just throwing in patches without either my or upstream's approval is rather harmful for everyone involved.  Getting approval will at least put it through some review process to make sure the patches don't break something else, which will benefit OLPC and its consumers and its upstream Fedora.  If an OLPC patch ends up breaking things, it makes both Mozilla and Fedora look bad.
Comment 5 Peter Robinson 2008-11-25 16:52:09 EST
OK, this has been highlighted this with Simon (via Danial) and I'll ask him to take that forward for upstream. I'm no real developer, more a Fedora packager working with jkatz and others to help OLPC get things as mainstream as possible and help with the packaging side of things so their dev's can develop.

Christopher, do you know what the Moz plans are with gio and what what extras libgnome gives to Fedora over mainline building without libgnome and friends, or if FF 3.1 or XL 1.9.1 will help with moving away from libgnome like gnome upstream is doing? Or even when there might be a 1.9.1 beta that they can test to see if that improves things :-)
Comment 6 Bug Zapper 2008-11-26 00:51:19 EST
This bug appears to have been reported against 'rawhide' during the Fedora 10 development cycle.
Changing version to '10'.

More information and reason for this action is here:
Comment 7 Peter Robinson 2008-11-26 15:22:40 EST
The OLPC Browser people are going to investigate further over the next couple of days to review requirements and file bugs upstream as necessary. I've filed a OLPC tracking bug which I've linked to the external bug tracker on this bug. I think we should leave this one open for tracking the Fedora requirements.
Comment 8 Bug Zapper 2009-06-09 05:54:56 EDT
This bug appears to have been reported against 'rawhide' during the Fedora 11 development cycle.
Changing version to '11'.

More information and reason for this action is here:
Comment 9 Peter Robinson 2009-08-03 08:16:12 EDT
Closing as it looks like its all OK with xulrunner 1.9.1 appart from the requirement on gnome-vfs and there's not much we can do about that atm.

Note You need to log in before you can comment on or make changes to this bug.