268314 - GNOME integration non-functional on Fedora Core 3 and newer

Status: VERIFIED FIXED

(Firefox :: Shell Integration, defect)

Description

[Brian Ryner (not reading)]

In Fedora Core 3 test3, and the release candidates that have been posted, GNOME
integration is non-functional with our release builds (tested 2004110712).

Symptoms:

- does not offer to become the default browser
- GNOME helper app/protocol associations are not used, including mailto
- mail toolbar button not present
- set as background image not present

What is happening is, our GNOME support library (libmozgnome) fails to load
because it links against liblinc.so.1.  This library was removed from the
distribution between FC3 test2 and test3.  So a clean install of FC3 will not
have this library present.

Note that the Firefox build included _with_ FC3 works ok because it is compiled
in the newer development environment.  It seems that "-llinc" used to be in the
link line for all of the GNOME libraries as returned by pkg-config, but no
longer is.

Comment 1

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Can we add the old liblinc.so.1 into our tarball?  We really don't have any
options, short of making two separate builds -- FC3 already requires that you
have the xorg-x11-deprecated-libs package installed for our builds to run.  In
the future we can try building on FC3, but I don't know whether those binaries
will run on earlier releases...

Comment 2

[Brian Ryner (not reading)]

Attachment: one way we could fix this

This patch would make us never pull in -llinc from the gnome-vfs / libgnome /
gconf pkg-config results.  We don't use any symbols from liblinc, so I _think_
this will work (and indeed, the library loads ok on both RH8 and FC3).

Alternately, we could hack ORBit-2.0.pc file on the build machine, or if
possible, update to a version of ORBit2 that's still binary-compatible but
doesn't pull in -llinc (this is the package that is causing -llinc to be pulled
in for anything that depends on ORBit).

Comment 3

[Brian Ryner (not reading)]

Or, we may just need to relnote as follows for FF 1.0:
--------------------------------------

Users of Fedora Core 3 will need to download and install:

http://download.fedora.redhat.com/pub/fedora/linux/core/2/i386/os/Fedora/RPMS/linc-1.0.3-3.1.i386.rpm

After installing the RPM, perform the following command in the directory you
installed firefox into (you will need write permission):

touch .autoreg

The next time you start Firefox, GNOME integration should be functional.
------------------------------------------

Comment 4

[Brian Ryner (not reading)]

Comment on attachment 165079 [details] [diff] [review]
one way we could fix this

dbaron, mscott and I had discussed the possibility of taking this fix for
Thunderbird 1.0.  What do you think?

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 165079 [details] [diff] [review]
one way we could fix this

sure

Comment 6

[sairuh (rarely reading bugmail)]

note that using trunk firefox builds, the pref for asking to make ffox the
default browser is gone (at least testing recent trunk builds on fedora core 2),
covered by bug 273439.

Comment 7

[sairuh (rarely reading bugmail)]

Tracy, how does this work for you during your smoketests? (I'll be upgrading
from fc2 sometime soon.)

Comment 8

[Adam Guthrie]

rhelmer says we can probably fix this by changing some of the tinderbox configurations. Is this something that we'd be willing to change for 1.5.0.* or 2.0? This could possibly give us some big wins judging from bryner's comment 0.

Comment 9

[Mike Beltzner [:beltzner, not reading bugmail]]

We're not going to block on this, but if the patch gets reviewed and baked, please nominate it for approval1.8.1

Comment 10

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

It might be good to use "s/-linc\>//" rather than just "s/-linc//", actually.

Comment 11

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: updated patch

Comment 12

[Robert Sayre]

changing the milestone because we have a recent patch

Comment 13

[Benjamin Smedberg]

Comment on attachment 231361 [details] [diff] [review]
updated patch

yick, ok

Comment 14

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Checked in to trunk.

Comment 15

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 231361 [details] [diff] [review]
updated patch

Would be good if somebody verifies this in the nightlies.

Comment 16

[Adam Guthrie]

I just grabbed a build from argo (2006080717) and this is still not working. :( Are we hitting the codepaths where we remove -llinc on that tinderbox?

Comment 17

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

It's not going to show up until a clobber build, I think.

But I also noticed I need to fix MOZ_GNOMEUI_LIBS too.  I'll check that in now.

Comment 18

[Adam Guthrie]

Latest tinderbox builds from argo look good. I can now check for/set as the default browser and set an image as the desktop background. ->VERIFIED

Comment 19

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 231361 [details] [diff] [review]
updated patch

a=drivers, go ahead and land on the branch

Comment 20

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

It would also be good if somebody (if possible) checks that this didn't regress these features on FC2.

Comment 21

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Checked in to MOZILLA_1_8_BRANCH.

Comment 22

[Adam Guthrie]

Verified FIXED using a current branch build from prometheus-vm (2006080817).

I was hoping for sexy GNOME-icon-goodness on dialog buttons, etc. like I see in my own builds, but I guess I'll have to file a separate bug for this.

Thanks for fixing this dbaron and bryner!