Should use standalone xulrunner glue and not rely on shell ordering to set MOZLIBPATH

Bug #427817 reported by Loïc Minier
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gnome-web-photo (Ubuntu)
Invalid
Wishlist
Unassigned

Bug Description

Binary package hint: gnome-web-photo

Hi

The shell wrapper to set MOZLIBPATH depends on shell ordering of xulrunner dirs to work which is fragile and probably broken. Alexander suggested that it should use the standalone glue instead.

It seems it might be using the standalone glue already in which case the shell wrapper can be dropped IIUC.

Bye

Revision history for this message
Alexander Sack (asac) wrote :

code seems to be ready for this. oddly the m4/libxul.m4 does not provide any --with switch ... so the confiugre.ac should use LIBXUL_INIT ([embedding])

in turn the .real hacks should be dropped.

Revision history for this message
Alexander Sack (asac) wrote :

m4/libxul.m4 does everything right, but then it just overrides the .pc files decisions like:

# FIXMEchpe: this isn't right. The pc file seems buggy, but until
# I can figure this out, do it like this:
libxul_cv_libdir="$($PKG_CONFIG --variable=sdkdir $libxul_cv_libxul_pkg)/bin"

libxul_cv_includes="$($PKG_CONFIG --cflags-only-I libxul-unstable libxul)"

i will check that.

Loïc Minier (lool)
Changed in gnome-web-photo (Ubuntu):
milestone: none → later
Revision history for this message
Micah Gersten (micahg) wrote :

Per discussion in bug 625801, we'll try to fix this for Natty.

Changed in gnome-web-photo (Ubuntu):
assignee: nobody → Chris Coulson (chrisccoulson)
importance: Undecided → Wishlist
milestone: later → natty-alpha-3
status: New → Triaged
Revision history for this message
Chris Coulson (chrisccoulson) wrote :

This is pretty difficult to do. It's currently using a lot of symbols exported from libxul which aren't available via XPCOM, and this makes it difficult to use pure standalone glue. In addition to that, it is currently linking against both the standalone and dependent xpcom glue libraries, which is pretty awful

Changed in gnome-web-photo (Ubuntu):
assignee: Chris Coulson (chrisccoulson) → nobody
Revision history for this message
Micah Gersten (micahg) wrote :

gnome-web-photo has been ported to webkit so this is no longer needed

Changed in gnome-web-photo (Ubuntu):
milestone: natty-alpha-3 → none
status: Triaged → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.