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 607627 - gjs-1.0.pc adds unnecessary libmozjs linkage to client apps
gjs-1.0.pc adds unnecessary libmozjs linkage to client apps
Status: RESOLVED OBSOLETE
Product: gjs
Classification: Bindings
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gjs-maint
gjs-maint
Depends on:
Blocks:
 
 
Reported: 2010-01-21 03:34 UTC by Christopher Halse Rogers
Modified: 2016-11-29 05:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Move @JS_PACKAGE@ to Requires.private (408 bytes, patch)
2010-01-21 03:34 UTC, Christopher Halse Rogers
none Details | Review

Description Christopher Halse Rogers 2010-01-21 03:34:17 UTC
Created attachment 151912 [details] [review]
Move @JS_PACKAGE@ to Requires.private

gjs-1.0.pc.in has a Requires on @JS_PACKAGE@, but users of libgjs don't actually need any of the symbols from libmozjs.so.

Moving @JS_PACKAGE@ to Requires.private means that pkg-config --cflags gjs-1.0 will add mozilla-js's CFLAGS, which clients need, but pkg-config --libs gjs-1.0 will not include -lmozjs.

Since Mozilla don't support libmozjs as a system library, causing unnecessary dependencies on it is annoying for packagers.

The attached trivial patch moves @JS_PACKAGE@ to Requires.private.  For added security, a Requires on pkg-config >= 0.22 is added, as this the version of pkg-config (from 2007) which correctly supports CFLAGS from Requires.private

(Blog post from the pkg-config maintainer describing the behaviour of Requires.private http://err.no/personal/blog/tech/2008-03-25-18-07_pkg-config,_sonames_and_Requires.private )
Comment 1 Philip Chimento 2016-11-29 05:16:38 UTC
This is obsolete, libmozjs is now in Requires.private already. Thanks for the patch anyway and sorry no-one committed it 7 years ago!