build: reorganise the build files

Shuffle the behaviour checks to the beginning of the processing.
Co-locate and hoist the options that the project supports, put custom
behavioural changes in a single location.

Author: compnerd

CMakeLists.txt

@@ -13,6 +13,27 @@ project(cmark-gfm)
 # version (the infixed `.gfm.`).
 set(PROJECT_VERSION 0.29.0.gfm.13)
 
+set(CMAKE_C_STANDARD 99)
+set(CMAKE_C_STANDARD_REQUIRED YES)
+set(CMAKE_C_EXTENSIONS NO)
+
+set(CMAKE_C_VISIBILITY_PRESET hidden)
+set(CMAKE_VISIBILITY_INLINES_HIDDEN TRUE)
+
+set(CMAKE_INCLUDE_CURRENT_DIR YES)
+
+option(CMARK_FUZZ_QUADRATIC "Build quadratic fuzzing harness" OFF)
+option(CMARK_LIB_FUZZER "Build libFuzzer fuzzing harness" OFF)
+option(CMARK_THREADING "Add locks around static accesses" OFF)
+
+if("${CMAKE_SOURCE_DIR}" STREQUAL "${CMAKE_BINARY_DIR}")
+    message(FATAL_ERROR "Do not build in-source.\nPlease remove CMakeCache.txt and the CMakeFiles/ directory.\nThen: mkdir build ; cd build ; cmake .. ; make")
+endif()
+if(NOT CMAKE_BUILD_TYPE)
+  set(CMAKE_BUILD_TYPE "Release" CACHE STRING
+  "Choose the type of build, options are: Debug Profile Release Asan Ubsan." FORCE)
+endif(NOT CMAKE_BUILD_TYPE)
+
 list(APPEND CMAKE_MODULE_PATH ${PROJECT_SOURCE_DIR}/cmake/modules)
 
 include(CheckFileOffsetBits)
@@ -58,24 +79,6 @@ add_compile_definitions($<$<CONFIG:Debug>:DEBUG>)
 
 add_compile_options($<$<AND:$<CONFIG:PROFILE>,$<COMPILE_LANGUAGE:C>>:-pg>)
 
-if("${CMAKE_SOURCE_DIR}" STREQUAL "${CMAKE_BINARY_DIR}")
-    message(FATAL_ERROR "Do not build in-source.\nPlease remove CMakeCache.txt and the CMakeFiles/ directory.\nThen: mkdir build ; cd build ; cmake .. ; make")
-endif()
-
-option(CMARK_LIB_FUZZER "Build libFuzzer fuzzing harness" OFF)
-option(CMARK_THREADING "Add locks around static accesses" OFF)
-
-set(CMAKE_C_STANDARD 99)
-set(CMAKE_C_STANDARD_REQUIRED YES)
-set(CMAKE_C_EXTENSIONS NO)
-
-set(CMAKE_C_VISIBILITY_PRESET hidden)
-set(CMAKE_VISIBILITY_INLINES_HIDDEN TRUE)
-
-set(CMAKE_INCLUDE_CURRENT_DIR YES)
-
-option(CMARK_FUZZ_QUADRATIC "Build quadratic fuzzing harness" OFF)
-
 if(CMARK_LIB_FUZZER)
   add_compile_options($<$<COMPILE_LANGUAGE:C>:-fsanitize-coverage=trace-pc-guard>)
 endif()
@@ -89,6 +92,8 @@ if(CMARK_FUZZ_QUADRATIC)
   add_link_options($<$<COMPILE_LANGUAGE:C>:-fsanitize=address>)
 endif()
 
+check_file_offset_bits()
+
 add_subdirectory(src)
 add_subdirectory(extensions)
 # TODO(compnerd) should this be enabled for MinGW, which sets CMAKE_SYSTEM_NAME
@@ -107,7 +112,3 @@ endif()
 export(TARGETS libcmark-gfm libcmark-gfm-extensions
   FILE cmark-gfmConfig.cmake)
 
-if(NOT CMAKE_BUILD_TYPE)
-  set(CMAKE_BUILD_TYPE "Release" CACHE STRING
-  "Choose the type of build, options are: Debug Profile Release Asan Ubsan." FORCE)
-endif(NOT CMAKE_BUILD_TYPE)

src/CMakeLists.txt

@@ -1,14 +1,6 @@
 configure_file(${CMAKE_CURRENT_SOURCE_DIR}/libcmark-gfm.pc.in
   ${CMAKE_CURRENT_BINARY_DIR}/libcmark-gfm.pc @ONLY)
 
-check_file_offset_bits()
-
-add_executable(cmark-gfm
-  ${PROJECT_SOURCE_DIR}/bin/main.c)
-target_link_libraries(cmark-gfm
-  libcmark-gfm
-  libcmark-gfm-extensions)
-
 add_library(libcmark-gfm
   arena.c
   blocks.c
@@ -53,6 +45,12 @@ set_target_properties(libcmark-gfm PROPERTIES
   SOVERSION ${PROJECT_VERSION}
   VERSION ${PROJECT_VERSION})
 
+add_executable(cmark-gfm
+  ${PROJECT_SOURCE_DIR}/bin/main.c)
+target_link_libraries(cmark-gfm
+  libcmark-gfm
+  libcmark-gfm-extensions)
+
 
 install(TARGETS cmark-gfm libcmark-gfm
   EXPORT cmark-gfm