[SCM] musescore/master: patch: put common flags into CMAKE_CXX_FLAGS
jcowgill at users.alioth.debian.org
jcowgill at users.alioth.debian.org
Mon Jul 11 10:37:19 UTC 2016
The following commit has been merged in the master branch:
commit a8e2e291d465df79e09a6ac20fa004ac1b2cf1c7
Author: James Cowgill <james410 at cowgill.org.uk>
Date: Thu Jul 7 22:29:57 2016 +0200
patch: put common flags into CMAKE_CXX_FLAGS
diff --git a/debian/patches/02-global-flags.patch b/debian/patches/02-global-flags.patch
new file mode 100644
index 0000000..86abffa
--- /dev/null
+++ b/debian/patches/02-global-flags.patch
@@ -0,0 +1,65 @@
+Description: Put common flags into CMAKE_CXX_FLAGS
+ Debhelper intentionally sets CMAKE_BUILD_TYPE to None so that optimization
+ flags are not added by cmake. Unfortunately musescore only enables --std=c++11
+ for the Debug and Release build types.
+ .
+ This patch moves the common flags (c++ standard, warnings, etc) to
+ CMAKE_CXX_FLAGS which gets used in all build types.
+ .
+ The flags in mtest are duplicated and inherited from the root CMakeLists.txt
+ anyway.
+Author: James Cowgill <jcowgill at debian.org>
+---
+This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -148,17 +148,20 @@ if ("${CMAKE_CXX_COMPILER_ID}" MATCHES "
+ endif ("${CMAKE_CXX_COMPILER_ID}" MATCHES "GNU")
+
+ if (APPLE)
+- set(CMAKE_CXX_FLAGS_DEBUG "-std=c++11 -fPIC -stdlib=libc++ -g -Wno-inconsistent-missing-override")
+- set(CMAKE_CXX_FLAGS_RELEASE "-std=c++11 -fPIC -stdlib=libc++ -O2 -DNDEBUG -DQT_NO_DEBUG -Wno-inconsistent-missing-override")
++ set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -fPIC -stdlib=libc++ -Wno-inconsistent-missing-override")
++ set(CMAKE_CXX_FLAGS_DEBUG "-g")
++ set(CMAKE_CXX_FLAGS_RELEASE "-O2 -DNDEBUG -DQT_NO_DEBUG")
+ else (APPLE)
+ if (MINGW)
+ # -mno-ms-bitfields see #22048
+- set(CMAKE_CXX_FLAGS_DEBUG "-std=c++11 -mno-ms-bitfields -g")
+- set(CMAKE_CXX_FLAGS_RELEASE "-std=c++11 -mno-ms-bitfields -O2 -DNDEBUG -DQT_NO_DEBUG")
++ set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -mno-ms-bitfields")
++ set(CMAKE_CXX_FLAGS_DEBUG "-g")
++ set(CMAKE_CXX_FLAGS_RELEASE "-O2 -DNDEBUG -DQT_NO_DEBUG")
+ set(CMAKE_EXE_LINKER_FLAGS "-Wl,--large-address-aware")
+ else (MINGW)
+- set(CMAKE_CXX_FLAGS_DEBUG "-std=c++11 -fPIC -fPIE -g")
+- set(CMAKE_CXX_FLAGS_RELEASE "-std=c++11 -fPIC -O2 -DNDEBUG -DQT_NO_DEBUG")
++ set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -fPIC")
++ set(CMAKE_CXX_FLAGS_DEBUG "-fPIE -g")
++ set(CMAKE_CXX_FLAGS_RELEASE "-O2 -DNDEBUG -DQT_NO_DEBUG")
+ endif (MINGW)
+ endif(APPLE)
+
+--- a/mtest/CMakeLists.txt
++++ b/mtest/CMakeLists.txt
+@@ -112,20 +112,6 @@ target_link_libraries(
+
+ set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${Qt5Widgets_EXECUTABLE_COMPILE_FLAGS}")
+
+-if (APPLE)
+- set(CMAKE_CXX_FLAGS_DEBUG "-std=c++0x -fPIC -stdlib=libc++ -g -Wno-inconsistent-missing-override")
+- set(CMAKE_CXX_FLAGS_RELEASE "-std=c++0x -fPIC -stdlib=libc++ -O2 -DNDEBUG -DQT_NO_DEBUG -Wno-inconsistent-missing-override")
+-else (APPLE)
+- if (MINGW)
+- # -mno-ms-bitfields see #22048
+- set(CMAKE_CXX_FLAGS_DEBUG "-std=gnu++0x -mno-ms-bitfields -g")
+- set(CMAKE_CXX_FLAGS_RELEASE "-std=gnu++0x -mno-ms-bitfields -O2 -DNDEBUG -DQT_NO_DEBUG")
+- else (MINGW)
+- set(CMAKE_CXX_FLAGS_DEBUG "-std=gnu++0x -fPIC -fPIE -g")
+- set(CMAKE_CXX_FLAGS_RELEASE "-std=gnu++0x -fPIC -O2 -DNDEBUG -DQT_NO_DEBUG")
+- endif (MINGW)
+-endif(APPLE)
+-
+
+ string(REPLACE ";" ";-I" INC "${QT_INCLUDES}")
+
diff --git a/debian/patches/series b/debian/patches/series
index f5ad673..41b63c2 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1 +1,2 @@
01-debundle-freetype.patch
+02-global-flags.patch
--
musescore packaging
More information about the pkg-multimedia-commits
mailing list