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 603653 - Require libffi.pc
Require libffi.pc
Status: RESOLVED FIXED
Product: gobject-introspection
Classification: Platform
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gobject-introspection Maintainer(s)
gobject-introspection Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2009-12-02 23:04 UTC by Colin Walters
Modified: 2015-02-07 16:45 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Require libffi.pc (2.85 KB, patch)
2009-12-02 23:04 UTC, Colin Walters
committed Details | Review
Add ffi cflags/libs/requires to exported .pc (3.00 KB, patch)
2009-12-04 15:14 UTC, Johan (not receiving bugmail) Dahlin
none Details | Review
Add ffi cflags/libs/requires to exported .pc (3.00 KB, patch)
2009-12-04 15:23 UTC, Johan (not receiving bugmail) Dahlin
committed Details | Review

Description Colin Walters 2009-12-02 23:04:14 UTC
We use it for girffi.h.  Remove the code to detect libffi without the .pc
file.  If you don't have it, get it.
Comment 1 Colin Walters 2009-12-02 23:04:16 UTC
Created attachment 148971 [details] [review]
Require libffi.pc
Comment 2 Johan (not receiving bugmail) Dahlin 2009-12-02 23:11:25 UTC
Comment on attachment 148971 [details] [review]
Require libffi.pc

Probably a good idea to kill the #ifdefs and makefile logic as well.
Comment 3 Colin Walters 2009-12-02 23:12:43 UTC
Attachment 148971 [details] pushed as 494c1c9 - Require libffi.pc
Comment 4 Johan (not receiving bugmail) Dahlin 2009-12-04 14:24:56 UTC
I reverted this patch for two reasons:

 * It breaks Ubuntu Hardy which we use for litl, it'll make it harder for us
   to track upstream as the the libffi package in hard lacks a .pc file.
 * Stefan Kost pointed out that it breaks Suse 11.2, which ships a ffi package
  based on the version included in gcc which lacks a .pc file.

I also think that there are no .pc files included in the Mac OS X version 
Which means that we have to live with that check for a little longer.
Comment 5 Stefan Sauer (gstreamer, gtkdoc dev) 2009-12-04 14:35:20 UTC
I filed a bug for gcc - they seemed to forgot then pc-file when importing the sources
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42275
Comment 6 Johan (not receiving bugmail) Dahlin 2009-12-04 15:14:48 UTC
Created attachment 149086 [details] [review]
Add ffi cflags/libs/requires to exported .pc

This is required as external projects such as gjs
is now using girffi and thus needs the ffi cflags
to be able to link.
Comment 7 Johan (not receiving bugmail) Dahlin 2009-12-04 15:23:44 UTC
The following fix has been pushed:
bf393f0 Add ffi cflags/libs/requires to exported .pc
Comment 8 Johan (not receiving bugmail) Dahlin 2009-12-04 15:23:49 UTC
Created attachment 149089 [details] [review]
Add ffi cflags/libs/requires to exported .pc

This is required as external projects such as gjs
are now using girffi and thus need the ffi cflags
to be able to link.
Comment 9 André Klapper 2015-02-07 16:45:43 UTC
[Mass-moving gobject-introspection tickets to its own Bugzilla product - see bug 708029. Mass-filter your bugmail for this message: introspection20150207 ]