Opened 10 years ago

Closed 6 years ago

#46571 closed defect (fixed)

xulrunner @1.9.2.16 does not work on Tiger

Reported by: csanchezdll Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.3.3
Keywords: haspatch, powerpc Cc: chrstphrchvz (Christopher Chavez)
Port: xulrunner

Description

xulrunner seems to have been broken in Tiger for some time. According to #28272, Tiger is unsupported. Also, mozjs17 and policykit compilation problems prevented xulrunner to build. With those fixed (#46567 and #45832) xulrunner can be get to build and run correctly on Tiger PPC.

  1. python in Tiger is 2.3.5, whereas xulrunner requires 2.4. Attached patch uses python27.
  2. freetype2 changed header location (see #41593).
  3. xulrunner decides which gfx headers to export based on default toolkit, but this is not consistent in gfx source code. As a result, it will not compile with --enable-default-toolkit=cairo-gtk2 if cairo has quartz support (as MacPorts version does). A small patch in gfxASurface.cpp fixes this (maybe it it should be tried to use cairo-quartz toolkit?).
  4. NSPR link functions (PR_FindSymbolAndLibrary and friends) are unable to locate symbols from linked libraries on Darwin PPC. This makes xulrunner fail on detecting pango version and use an incorrect mechanism for setting fontmap property. This lead to runtime failed assertions:

GLib-GObject:ERROR:gobject.c:4270:g_weak_ref_set: assertion failed: (weak_locations != NULL)

As pango port is well above version 124.04, attached patch forces xulrunner to assumes a higher version instead of checking.

  1. Fixed in-tree libffi as per #21401

I reckon some of those should be applied upstream, but I woulld not expect new releases of 1.9 series, and 2.0 break PPC support. So I think it is sensible to apply them at MacPorts level.

Attachments (5)

Portfile.diff (1.2 KB) - added by csanchezdll 10 years ago.
patch-config-system-headers.diff (486 bytes) - added by csanchezdll 10 years ago.
patch-gfx-thebes-src-gfxASurface.cpp.diff (316 bytes) - added by csanchezdll 10 years ago.
patch-gfx-thebes-src-gfxPangoFonts.cpp.diff (641 bytes) - added by csanchezdll 10 years ago.
patch-js-ctypes-libffi-src-powerpc-fde.patch (1.2 KB) - added by csanchezdll 10 years ago.

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by csanchezdll

Attachment: Portfile.diff added

Changed 10 years ago by csanchezdll

Changed 10 years ago by csanchezdll

Changed 10 years ago by csanchezdll

Changed 10 years ago by csanchezdll

comment:1 Changed 6 years ago by chrstphrchvz (Christopher Chavez)

Cc: chrstphrchvz added

comment:2 Changed 6 years ago by chrstphrchvz (Christopher Chavez)

Resolution: fixed
Status: newclosed

In e5df202ea749d98d85ac561c4f85eea5e51dbd69/macports-ports (master):

xulrunner[-devel]: delete ports

Ports are outdated, nomaintainer, and discontinued upstream.

openvrml: remove +mozilla_plugin variant due to XULRunner dependency

Closes: #29556
Closes: #36908
Closes: #39639
Closes: #46571

Note: See TracTickets for help on using tickets.