Review Request 121982: Do not unset cache variables in KDEInstallDirs.
Kevin Kofler
kevin.kofler at chello.at
Sun Jan 11 12:59:35 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121982/#review73738
-----------------------------------------------------------
Ship it!
Works for me, the chain build succeeded.
- Kevin Kofler
On Jan. 11, 2015, 11:12 vorm., Alex Merry wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121982/
> -----------------------------------------------------------
>
> (Updated Jan. 11, 2015, 11:12 vorm.)
>
>
> Review request for Extra Cmake Modules and Kevin Kofler.
>
>
> Repository: extra-cmake-modules
>
>
> Description
> -------
>
> Cache variables such as CMAKE_INSTALL_BINDIR may be used by other
> modules included from parallel parts of the tree, so we should not touch
> them. We still override them in the runtime environment, but this will
> not interfere with parallel subtrees of the project.
>
> As part of this, the order of precedence of variables specified on the
> command line is changed, so that KDE_INSTALL_* is considered first
> (although it is still considered "undefined" in the documentation). This
> means that if you only specify CMAKE_INSTALL_BINDIR, that will be used
> by both KDEInstallDirs and GNUInstallDirs, but if you specify both that
> and KDE_INSTALL_BINDIR, KDEInstallDirs will use KDE_INSTALL_BINDIR
> instead. This is probably not too useful, but seems more useful than
> any other arrangement.
>
> BUG: 342717
>
>
> Diffs
> -----
>
> kde-modules/KDEInstallDirs.cmake 7d592afcbfd87f08650428754da72e70a9b70155
>
> Diff: https://git.reviewboard.kde.org/r/121982/diff/
>
>
> Testing
> -------
>
> `make test` - all tests pass.
>
> Kevin: can you please test this with Calamares?
>
>
> Thanks,
>
> Alex Merry
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20150111/c027e445/attachment.html>
More information about the Kde-buildsystem
mailing list