Review Request 118498: Make ECM language-independent again, but make the tests use C
Commit Hook
null at kde.org
Tue Jun 17 18:17:52 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118498/#review60308
-----------------------------------------------------------
This review has been submitted with commit 3afc8f9875979e57cce97edd68a5971d6771e097 by Alex Merry to branch master.
- Commit Hook
On June 15, 2014, 4:13 p.m., Alex Merry wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118498/
> -----------------------------------------------------------
>
> (Updated June 15, 2014, 4:13 p.m.)
>
>
> Review request for Build System, Extra Cmake Modules, KDE Frameworks, Christophe Giboudeaux, Nicolás Alvarez, and Allen Winter.
>
>
> Repository: extra-cmake-modules
>
>
> Description
> -------
>
> Provide an option to exclude the tests
>
> While the tests in ECM are not "built" as such (at least, not until they
> are run), disabling the tests might be desirable to avoid the compiler
> checks and to make the whole build process architecture-independent.
>
> Make ECM language-independent again, but make the tests use C
>
> Setting the language for ECM's project() call to C had unanticipated
> side-effects - notably that the installed version file required the
> architecture to match the one used at build time.
>
> Instead, we make the tests a sub-project, setting up C as the language
> there (since most of the tests do use C, albeit slightly indirectly).
>
>
> Diffs
> -----
>
> CMakeLists.txt df0759f18f1ed091b43128c1c5844aead560fe5b
> tests/CMakeLists.txt dfcc252a4e723d7376f87153adcd23d00dbc7846
>
> Diff: https://git.reviewboard.kde.org/r/118498/diff/
>
>
> Testing
> -------
>
> Configures properly, tests pass. Visually inspected ECMConfigVersion.cmake file to see that it will return before the architecture check.
>
>
> Thanks,
>
> Alex Merry
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20140617/50366084/attachment.html>
More information about the Kde-buildsystem
mailing list