From 70367023241432723de59ff777da3c31e6b1caf6 Mon Sep 17 00:00:00 2001 From: kervala Date: Mon, 5 Dec 2016 14:06:48 +0100 Subject: [PATCH] Merge with develop --- code/CMakeModules/PCHSupport.cmake | 17 +++++++---------- code/CMakeModules/nel.cmake | 10 ++++++++++ .../client/ryzom_installer/src/configfile.cpp | 5 +++++ dist/snap/snapcraft.yaml | 7 +++++-- 4 files changed, 27 insertions(+), 12 deletions(-) diff --git a/code/CMakeModules/PCHSupport.cmake b/code/CMakeModules/PCHSupport.cmake index 45e1aa766..f56626da1 100644 --- a/code/CMakeModules/PCHSupport.cmake +++ b/code/CMakeModules/PCHSupport.cmake @@ -28,15 +28,12 @@ ELSE() ENDIF() MACRO(APPEND_DEFINITION _NAME _VAL) - # value is not empty - IF(${_VAL}) - IF(CMAKE_VERSION VERSION_LESS "2.8.12") - # don't support logical expressions, append definition - LIST(APPEND ${_NAME} "-D${_VAL}") - ELSE() - # support logical expressions, use them - LIST(APPEND ${_NAME} "$<$:-D$>") - ENDIF() + IF(CMAKE_VERSION VERSION_LESS "2.8.12") + # don't support logical expressions, append definition + LIST(APPEND ${_NAME} "-D${_VAL}") + ELSE() + # support logical expressions, use them + LIST(APPEND ${_NAME} "$<$:-D$>") ENDIF() ENDMACRO() @@ -197,7 +194,7 @@ MACRO(PCH_SET_COMPILE_FLAGS _target) LIST(APPEND _FLAGS "${item}") ENDFOREACH() ENDIF() - + GET_DIRECTORY_PROPERTY(_DIRECTORY_DEFINITIONS DIRECTORY ${CMAKE_SOURCE_DIR} DEFINITIONS) IF(_DIRECTORY_DEFINITIONS) diff --git a/code/CMakeModules/nel.cmake b/code/CMakeModules/nel.cmake index 08f2f120e..88c1a8538 100644 --- a/code/CMakeModules/nel.cmake +++ b/code/CMakeModules/nel.cmake @@ -832,10 +832,17 @@ MACRO(NL_SETUP_BUILD) ELSE() # Always force -mmacosx-version-min to override environement variable IF(CMAKE_OSX_DEPLOYMENT_TARGET) + IF(CMAKE_OSX_DEPLOYMENT_TARGET VERSION_LESS "10.7") + MESSAGE(FATAL_ERROR "Minimum target for OS X is 10.7 but you're using ${CMAKE_OSX_DEPLOYMENT_TARGET}") + ENDIF() SET(PLATFORM_LINKFLAGS "${PLATFORM_LINKFLAGS} -Wl,-macosx_version_min,${CMAKE_OSX_DEPLOYMENT_TARGET}") ENDIF() ENDIF() + # use libc++ under OX X to be able to use new C++ features (and else it'll use GCC 4.2.1 STL) + # minimum target is now OS X 10.7 + ADD_PLATFORM_FLAGS("-stdlib=libc++") + SET(PLATFORM_LINKFLAGS "${PLATFORM_LINKFLAGS} -Wl,-headerpad_max_install_names") IF(HAVE_FLAG_SEARCH_PATHS_FIRST) @@ -852,6 +859,9 @@ MACRO(NL_SETUP_BUILD) ENDIF() ENDIF() + # use c++0x standard to use std::unique_ptr and std::shared_ptr + ADD_PLATFORM_FLAGS("-std=c++0x") + ADD_PLATFORM_FLAGS("-D_REENTRANT") # hardening diff --git a/code/ryzom/tools/client/ryzom_installer/src/configfile.cpp b/code/ryzom/tools/client/ryzom_installer/src/configfile.cpp index 592b8e0a2..f76ba4771 100644 --- a/code/ryzom/tools/client/ryzom_installer/src/configfile.cpp +++ b/code/ryzom/tools/client/ryzom_installer/src/configfile.cpp @@ -715,8 +715,13 @@ int CConfigFile::compareInstallersVersion() const QString newVersion = QApplication::applicationVersion(); +#if (QT_VERSION >= QT_VERSION_CHECK(5, 6, 0)) QVersionNumber installedVer = QVersionNumber::fromString(installedVersion); QVersionNumber newVer = QVersionNumber::fromString(newVersion); +#else + QString installedVer = installedVersion; + QString newVer = newVersion; +#endif // same version if (newVer == installedVer) return 0; diff --git a/dist/snap/snapcraft.yaml b/dist/snap/snapcraft.yaml index 659a58a7d..7c0bd1813 100644 --- a/dist/snap/snapcraft.yaml +++ b/dist/snap/snapcraft.yaml @@ -42,11 +42,14 @@ parts: - -DRYZOM_GAMES_PREFIX=bin build-packages: - zlib1g-dev - - libpng-dev + - libpng12-dev - libjpeg8-dev - qttools5-dev-tools + - libxml2-dev + - libssl-dev + - libqt5opengl5-dev stage-packages: - libqt5widgets5 - + - libqt5opengl5 after: [desktop-qt5]