----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117619/#review56498 -----------------------------------------------------------
This review has been submitted with commit 035c6fa9390ecb66422ee684be71eea692989009 by Alex Merry to branch master. - Commit Hook On April 21, 2014, 9:49 p.m., Alex Merry wrote: > > ----------------------------------------------------------- > 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. > > > 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 > ----- > > 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 > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
