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 766746 - Use GNUInstallDirs cmake module for controlling install directories
Use GNUInstallDirs cmake module for controlling install directories
Status: RESOLVED INCOMPLETE
Product: geary
Classification: Other
Component: build
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Geary Maintainers
Geary Maintainers
Depends on: 766196
Blocks:
 
 
Reported: 2016-05-21 11:31 UTC by Julian Ospald
Modified: 2018-02-07 17:51 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Use GNUInstallDirs cmake module (6.26 KB, patch)
2016-05-21 11:31 UTC, Julian Ospald 		none Details | Review

Description Julian Ospald 2016-05-21 11:31:37 UTC 
Created attachment 328311 [details] [review]
Use GNUInstallDirs cmake module

This is useful if 'bin' and 'share' don't have the same prefix,
allowing things like:
* /usr/share/geary/...
* /usr/<host-prefix>/bin/geary
Comment 1 Michael Gratton 2016-05-30 03:55:29 UTC 
Hi Julian, thanks for the patch.

Is GNUInstallDirs part of the standard CMake distribution, or is it an add-on? If so, which distros ship it?
Comment 3 Michael Gratton 2016-07-01 05:12:15 UTC 
Okay, so it is included by CMake 3.0, which is what Debian stable is currently shipping.

I'll commit this to master once 766196 has landed, since there will be a few conflicts in the CMakeList files.
Comment 4 Michael Gratton 2017-02-07 14:22:34 UTC 
This is getting a bit high-risk to land so close to 0.12, so bumping to 0.13.
Comment 5 Federico Bruni 2017-12-04 06:55:44 UTC 
Mike, is this going to be committed  now that 0.12 has been released?

We don't know when the build system will be migrated to Meson: bug #777044
Comment 6 Michael Gratton 2017-12-12 11:01:57 UTC 
Julian, apologies for the delay in getting this patch committed. We have recently committed a new meson-based build system, does Meson out of the box support the kinds of customisations you were looking for with this patch?
Comment 7 Michael Gratton 2018-02-07 07:07:43 UTC 
Closing this bug report as no further information has been provided. Please feel free to reopen this bug report if you can provide the information that was asked for in a previous comment.
Thanks!
Comment 8 Julian Ospald 2018-02-07 17:51:11 UTC 
I don't use geary anymore and the contribution workflow is definitely too slow for me to care or investigate.