]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
cmake: set CMAKE_BUILD_TYPE only if .git exists 42129/head
authorKefu Chai <kchai@redhat.com>
Thu, 1 Jul 2021 07:24:50 +0000 (15:24 +0800)
committerKefu Chai <kchai@redhat.com>
Thu, 1 Jul 2021 08:57:20 +0000 (16:57 +0800)
distros intend to fine tune the CFLAGS and CXXFLAGS by themselves, see

- https://git.alpinelinux.org/abuild/tree/abuild.conf
- https://wiki.archlinux.org/title/CMake_package_guidelines#CMake_undesired_behaviors
- https://github.com/Debian/debhelper/blob/5d1bb29841043d8e47ebbdd043e6cd086cad508e/lib/Debian/Debhelper/Buildsystem/cmake.pm#L16

so instead of setting a CMAKE_BUILD_TYPE when building from a
dist tarball, let's just leave it empty.

Signed-off-by: Kefu Chai <kchai@redhat.com>
src/CMakeLists.txt

index 760a2ceb0ca0d23802e25e80b2ccaa16dd4e9ba4..6f35e87f90787103ed21e1774b8bc73f1a13076d 100644 (file)
@@ -156,12 +156,8 @@ elseif(CMAKE_CXX_COMPILER_ID STREQUAL Clang)
   endif()
 endif(CMAKE_CXX_COMPILER_ID STREQUAL GNU)
 
-if(NOT CMAKE_BUILD_TYPE)
-  if(EXISTS "${CMAKE_SOURCE_DIR}/.git")
-    set(default_build_type "Debug")
-  else()
-    set(default_build_type "RelWithDebInfo")
-  endif()
+if(DEFINED CMAKE_BUILD_TYPE OR EXISTS "${CMAKE_SOURCE_DIR}/.git")
+  set(default_build_type "Debug")
   set(CMAKE_BUILD_TYPE "${default_build_type}" CACHE
       STRING "Default BUILD_TYPE is Debug, other options are: RelWithDebInfo, Release, and MinSizeRel." FORCE)
 endif()