Wireshark-commits: [Wireshark-commits] master 4c1690a: CMake: require at least CMake 3.5
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=4c1690ac4783c7334462b4140a9c41526edbae44
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark
Commits:
4c1690a by Peter Wu (peter@xxxxxxxxxxxxx):
CMake: require at least CMake 3.5
CMake 3.11 with the Ninja generator started complaining about CMP0058
related to ui/qt/CMakeFiles/qtui_autogen.dir/RCCstock_iconsInfo.cmake
amd other files (AUTORCC). While the policy could be set explicitly,
let's try to modernize the CMake configuration:
- Drop CMP0042, if this gives issues with macOS, then it must be solved
in a different way using non-deprecated methods.
- Drop CMP0054 and ensure that all if("${foo}") and if(${foo}) are
converted to if(foo).
- Remove string comparison against "-NOTFOUND", it already evaluates to
false in an if condition.
- Use CXX_STANDARD/CXX_STANDARD_REQUIRED for Qt 5.7 and newer.
- Assume that copy_if_different can accept multiple sources (CMake 3.5).
- Consistency: Out of the 60 CMake 3.11 FindXxx.cmake files that use
find_library, 34 contain "XXX_LIBRAR" while 16 contain "Xxx_LIBRAR".
Let's assume uppercase variables (now custom MaxMindDB include dirs
are correctly used).
CMake 3.5 was chosen as the next version because of its wide support.
Ubuntu 14.04 ships with cmake3 3.5.1, Debian jessie-backports has 3.6.2,
EPEL for CentOS/RHEL6 includes cmake3 3.6.1 and SLES12 SP2 has 3.5.
Change-Id: I2fa7b94bf8cc78411f414987d17bab3a33dfb360
Reviewed-on: https://code.wireshark.org/review/27444
Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
Actions performed:
from e21da73 CMake: fix Git binary parameter
adds 4c1690a CMake: require at least CMake 3.5
Summary of changes:
CMakeLists.txt | 125 +++++++++++++---------------------------
debian/control | 2 +-
epan/CMakeLists.txt | 9 +--
epan/dissectors/CMakeLists.txt | 2 +-
packaging/nsis/CMakeLists.txt | 4 +-
packaging/rpm/wireshark.spec.in | 2 +-
packaging/wix/CMakeLists.txt | 4 +-
tools/debian-setup.sh | 6 +-
ui/qt/CMakeLists.txt | 34 ++++-------
9 files changed, 63 insertions(+), 125 deletions(-)
