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 674010 - Fix libmusicbrainz .pc installation path
Fix libmusicbrainz .pc installation path
Status: RESOLVED FIXED
Product: jhbuild
Classification: Infrastructure
Component: module sets
unspecified
Other All
: Normal normal
: ---
Assigned To: Jhbuild maintainers
Jhbuild QA
Depends on:
Blocks:
 
 
Reported: 2012-04-12 18:55 UTC by Marc-Andre Lureau
Modified: 2012-05-21 12:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
cmake: allow substituion of libsuffix variable (1.10 KB, patch)
2012-04-12 18:55 UTC, Marc-Andre Lureau
needs-work Details | Review
core-deps-3.[4,6]: fix libmusicbrainz .pc installation (1.66 KB, patch)
2012-04-12 18:55 UTC, Marc-Andre Lureau
committed Details | Review
cmake: allow substituion of libsuffix variable (1.28 KB, patch)
2012-05-06 15:54 UTC, Marc-Andre Lureau
committed Details | Review

Description Marc-Andre Lureau 2012-04-12 18:55:29 UTC
Without specifying LIB_SUFFIX, the .pc ends up installed to $prefix/lib, even if jhbuild is set to use_lib64.
Comment 1 Marc-Andre Lureau 2012-04-12 18:55:32 UTC
Created attachment 211951 [details] [review]
cmake: allow substituion of libsuffix variable

libmusicbrainz uses a libsuffix value, "32" or "64"
Comment 2 Marc-Andre Lureau 2012-04-12 18:55:38 UTC
Created attachment 211952 [details] [review]
core-deps-3.[4,6]: fix libmusicbrainz .pc installation

Use custom cmake args with ${libsuffix} to fix installation path of .pc
Comment 3 Craig Keogh 2012-04-16 05:02:58 UTC
Review of attachment 211951 [details] [review]:

Thank you for the patch. On a 32bit system, your patch would cause some of libmusicbrainz to be installed to /opt/gnome/lib32. On 32bit systems, JHBuild packages install in /opt/gnome/lib
Comment 4 Marc-Andre Lureau 2012-05-06 15:54:17 UTC
Created attachment 213553 [details] [review]
cmake: allow substituion of libsuffix variable

libmusicbrainz uses a libsuffix value, "32" or "64"
Comment 5 Craig Keogh 2012-05-21 12:57:03 UTC
Comment on attachment 213553 [details] [review]
cmake: allow substituion of libsuffix variable

Thank you for your patch. I reworked [1] eval_args in an earlier commit. So I changed your patch to call the parent class eval_args, and committed here:
http://git.gnome.org/browse/jhbuild/commit/?id=1a24440667b0696096a4ce7800ada2b80a9e8e89

[1] http://git.gnome.org/browse/jhbuild/commit/?id=4b85c395c57cf16636e54901e2658a87b3956aa9
Comment 6 Craig Keogh 2012-05-21 12:57:30 UTC
Comment on attachment 211952 [details] [review]
core-deps-3.[4,6]: fix libmusicbrainz .pc installation

Committed.
http://git.gnome.org/browse/jhbuild/commit/?id=36b2a50fa9374dab58f2f3cc8b55cf873198f356