After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 402187 - [PATCH] Missing linked libraries for browser plugin
[PATCH] Missing linked libraries for browser plugin
Status: RESOLVED DUPLICATE of bug 351266
Product: totem
Classification: Core
Component: Browser plugin (obsolete)
2.17.x
Other Linux
: Normal normal
: ---
Assigned To: totem-browser-maint
totem-browser-maint
Depends on:
Blocks:
 
 
Reported: 2007-01-29 18:58 UTC by Saleem Abdulrasool
Modified: 2007-02-07 07:52 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18


Attachments
totem-2.17.5-browser-plugin.patch (2.13 KB, patch)
2007-01-29 18:59 UTC, Saleem Abdulrasool
rejected Details | Review

Description Saleem Abdulrasool 2007-01-29 18:58:09 UTC
The browser plugins need libstartup-notification, which is missing from the linking flags passed to them.  Additionally, the XPCOM links are missing as well.  The attached patch should fix the issues.
Comment 1 Saleem Abdulrasool 2007-01-29 18:59:00 UTC
Created attachment 81460 [details] [review]
totem-2.17.5-browser-plugin.patch
Comment 2 Bastien Nocera 2007-01-29 22:48:11 UTC
That looks wrong. The browser plugins certainly don't need to be linking to libstartup-notification, given that they don't use it (totem-plugin-viewer is, and does), and the rest of the "missing" XPCOM links should be resolved at run-time.

What were the actual problems encountered?
Comment 3 Saleem Abdulrasool 2007-01-30 07:54:27 UTC
Hmm, I think I made the libstartup-notification a bit too wide in application, however it is needed for totem-plugin-viewer.  It not only includes the header, but calls functions from it (sn_launcher_context_get_startup_id fex).

As for the XPCOM links, it results in the browser plugin failing to run and crashing Firefox.
Comment 4 Saleem Abdulrasool 2007-01-30 07:56:20 UTC
A quick second glance at the patch reveals that the SN_{LIBS,CFLAGS} are being added only to the totem-plugin-viewer, which is the component which requires it.
Comment 5 Bastien Nocera 2007-01-30 10:17:20 UTC
(In reply to comment #3)
> Hmm, I think I made the libstartup-notification a bit too wide in application,
> however it is needed for totem-plugin-viewer.  It not only includes the header,
> but calls functions from it (sn_launcher_context_get_startup_id fex).

Are you compiling the GTK+ only version? If so, it looks like it's missing from the GTK+ checks. The GNOME version is already pulling it through gnome-desktop

> As for the XPCOM links, it results in the browser plugin failing to run and
> crashing Firefox.

Please file a bug against the browser plugin, and quote which version of Mozilla/Seamonkey/etc. you're compiling against, and against which you're running.
Comment 6 Christian Persch 2007-01-30 12:12:44 UTC
I guess the xpcom lib problem is bug 351266 ?
Comment 7 Saleem Abdulrasool 2007-02-07 07:52:41 UTC
Yeah, its the same as bug #351266 I do believe.  Marking this as a duplicate as such.

*** This bug has been marked as a duplicate of 351266 ***