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 550231 - missing dep on libffi in pygobject-2.0.pc
missing dep on libffi in pygobject-2.0.pc
Status: RESOLVED FIXED
Product: pygobject
Classification: Bindings
Component: general
2.15.x
Other Linux
: Normal normal
: ---
Assigned To: Nobody's working on this now (help wanted and appreciated)
Python bindings maintainers
Depends on:
Blocks:
 
 
Reported: 2008-09-01 14:34 UTC by Götz Waschk
Modified: 2009-05-17 15:09 UTC
See Also:
GNOME target: ---
GNOME version: 2.23/2.24


Attachments
fix pkgconfig files (1.65 KB, patch)
2008-09-01 14:36 UTC, Götz Waschk
committed Details | Review
Use Requires.private (729 bytes, patch)
2009-05-12 07:35 UTC, Josselin Mouette
committed Details | Review

Description Götz Waschk 2008-09-01 14:34:57 UTC
If pygobject was built with ffi support, -lffi is manually added to the pkgconfig file. It would be better to add libffi as requirement to the pkgconfig file, then this dep is caught by rpm automatically.
Comment 1 Götz Waschk 2008-09-01 14:36:51 UTC
Created attachment 117770 [details] [review]
fix pkgconfig files

This is how I have fixed this in the Mandriva package.
Comment 2 Paul Pogonyshev 2008-12-06 19:58:55 UTC
Does it mean libffi become mandatory?  Or is it still possible to build without libffi?
Comment 3 Götz Waschk 2008-12-07 12:08:09 UTC
No, libffi is still optional. But the pkg-config file should properly reflect this if ffi is enabled.
Comment 4 Paul Pogonyshev 2009-04-14 21:21:59 UTC
OK, hopefully it works as advertised ;)

Sending        ChangeLog
Sending        configure.ac
Sending        pygobject-2.0-uninstalled.pc.in
Sending        pygobject-2.0.pc.in
Transmitting file data ....
Committed revision 1065.
Comment 5 Josselin Mouette 2009-05-11 12:35:22 UTC
Unless there is a point linking all packages using pygobject directly against libffi (and I don’t think there is), it should be in Requires.private, not in Requires.
Comment 6 Paul Pogonyshev 2009-05-11 20:30:54 UTC
I'm absolutely clueless about packaging.  Can you provide a patch?
Comment 7 Josselin Mouette 2009-05-12 07:35:22 UTC
Created attachment 134466 [details] [review]
Use Requires.private

Something like this should do the trick.
Comment 8 Paul Pogonyshev 2009-05-12 17:43:43 UTC
Götz: does it still work for your original purpose after this correction?
Comment 9 Götz Waschk 2009-05-13 08:20:20 UTC
No, with the patch applied I get this error when building pygtk:

checking for PYGOBJECT... configure: error: Package requirements (pygobject-2.0 >= 2.16.1) were not met:

Package libffi was not found in the pkg-config search path.
Perhaps you should add the directory containing `libffi.pc'
to the PKG_CONFIG_PATH environment variable
Package 'libffi', required by 'PyGObject', not found

Consider adjusting the PKG_CONFIG_PATH environment variable if you
installed software in a non-standard prefix.


Deps in Requires.private are not picked up by rpm but they are still needed by pkg-config (it seems). So is this a bug in the automatic rpm dependancy generation?
Comment 10 Götz Waschk 2009-05-13 08:35:10 UTC
I think we can settle on a rpm automatic deps or a pkg-config problem, so it is not a bug in pygobject if the ffi dep is in Requires.private. pygtk is fine if it is not linked with -lffi.
Comment 11 Josselin Mouette 2009-05-13 21:52:31 UTC
(In reply to comment #9)
> Deps in Requires.private are not picked up by rpm but they are still needed by
> pkg-config (it seems). So is this a bug in the automatic rpm dependancy
> generation?

Yes; for dependency generation purposes, Requires.private should be treated exactly as Requires.
Comment 12 Paul Pogonyshev 2009-05-13 21:55:27 UTC
OK, as I understand the correction patch from Josselin is correct and should be applied, and further fixes should go into rpm dependency finder, right?
Comment 13 Paul Pogonyshev 2009-05-17 15:09:16 UTC
I'll commit this as we are going to release 2.18 and as I understood the patch is correct.