Review Request 117619: Make ecm_setup_version interact well with CMP0048

Alex Merry alex.merry at kde.org
Mon Apr 21 21:49:37 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117619/
-----------------------------------------------------------

(Updated April 21, 2014, 9:49 p.m.)


Review request for Build System, Extra Cmake Modules and KDE Frameworks.


Changes
-------

Clarify docs.


Repository: extra-cmake-modules


Description
-------

Make ecm_setup_version interact well with CMP0048

When CMake policy CMP0048 (CMake 3.0) is set to NEW, the project()
command is meant to manage the project's version variables. We therefore
do not set the PROJECT_VERSION variables in this case.

To make sure projects do not have to specify their version in multiple
places, this also allows the keyword "PROJECT" to be passed to
ecm_setup_version instead of an actual version number. In this case, the
version passed to project() will be used.


Diffs (updated)
-----

  modules/ECMSetupVersion.cmake 3ae6fa8f73357f5958de9f54adf083bf6de03df0 
  modules/ECMVersionHeader.h.in 19eb96c53d3b518df6b364e249707f12892b6893 
  tests/CMakeLists.txt e464a0305bd71364463c3132103ffe02dcb94eb6 
  tests/ECMSetupVersionTest/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_explicit_header/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_explicit_header/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_explicit_header_abspath/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_explicit_header_abspath/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_explicit_simple/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_explicit_simple/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_explicit_soversion/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_explicit_soversion/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_explicit_version_file/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_explicit_version_file/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_explicit_version_file_abspath/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_explicit_version_file_abspath/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_header/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_header/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_header_abspath/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_header_abspath/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_header_prefix/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_header_prefix/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_simple/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_simple/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_simple_prefix/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_simple_prefix/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_soversion/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_soversion/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_soversion_prefix/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_soversion_prefix/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_version_file/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_version_file/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_version_file_abspath/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/new_project_version_file_abspath/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/old_header/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/old_header/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/old_header_abspath/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/old_header_abspath/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/old_simple/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/old_simple/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/old_soversion/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/old_soversion/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/old_version_file/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/old_version_file/main.c PRE-CREATION 
  tests/ECMSetupVersionTest/old_version_file_abspath/CMakeLists.txt PRE-CREATION 
  tests/ECMSetupVersionTest/old_version_file_abspath/main.c PRE-CREATION 

Diff: https://git.reviewboard.kde.org/r/117619/diff/


Testing
-------

Bundled autotests pass. Freshly configured, built and installed KArchive - version header file and KF5ArchiveConfigVersion.cmake correct by inspection. KEmoticons then configured successfully (finding KF5Archive >= 4.96.0).


Thanks,

Alex Merry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140421/a4c67c5a/attachment.html>


More information about the Kde-frameworks-devel mailing list