Skip to content

Commit

Permalink
[CMake] Use target_compile_definitions to avoid affecting global defi…
Browse files Browse the repository at this point in the history
…nitions (#121)
  • Loading branch information
past-due authored and stevengj committed Apr 27, 2018
1 parent b945edd commit fe3f6bd
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
cmake_minimum_required (VERSION 2.8)
cmake_minimum_required (VERSION 2.8.12)

include (utils.cmake)

Expand All @@ -13,10 +13,6 @@ set(SO_MAJOR 2)
set(SO_MINOR 1)
set(SO_PATCH 1)

add_definitions (
-DUTF8PROC_EXPORTS
)

if (NOT MSVC)
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -O2 -std=c99 -pedantic -Wall")
endif ()
Expand All @@ -26,6 +22,8 @@ add_library (utf8proc
utf8proc.h
)

target_compile_definitions(utf8proc PRIVATE "UTF8PROC_EXPORTS")

set_target_properties (utf8proc PROPERTIES
POSITION_INDEPENDENT_CODE ON
VERSION "${SO_MAJOR}.${SO_MINOR}.${SO_PATCH}"
Expand Down

0 comments on commit fe3f6bd

Please sign in to comment.