Skip to content

Commit f68b89b

Browse files
committed
- Back off CMake version requirements.
- Fix plugin extension on OSX (it is .so, not .dylib) - add SKIP_TESTS to skip compiling test
1 parent 91ad315 commit f68b89b

File tree

4 files changed

+17
-14
lines changed

4 files changed

+17
-14
lines changed

CMakeLists.txt

+2-6
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,8 @@ SET(CPACK_PACKAGE_VERSION_MINOR 1)
1111
SET(CPACK_PACKAGE_VERSION_PATCH 0)
1212
SET(CPACK_PACKAGE_VERSION "${CPACK_PACKAGE_VERSION_MAJOR}.${CPACK_PACKAGE_VERSION_MINOR}.${CPACK_PACKAGE_VERSION_PATCH}")
1313

14-
#Minimum required version is Cmake 2.8.8, since we need to build object libraries
15-
IF(WIN32)
16-
CMAKE_MINIMUM_REQUIRED(VERSION 2.8.8 FATAL_ERROR)
17-
ELSE()
18-
CMAKE_MINIMUM_REQUIRED(VERSION 2.8.8 FATAL_ERROR)
19-
ENDIF()
14+
CMAKE_MINIMUM_REQUIRED(VERSION 2.8 FATAL_ERROR)
15+
2016
IF(COMMAND CMAKE_POLICY)
2117
SET(OLD_POLICIES CMP0026 CMP0042 CMP0045)
2218
SET(NEW_POLICIES CMP0003 CMP0022 CMP0023)

include/ma_global.h

-2
Original file line numberDiff line numberDiff line change
@@ -1066,8 +1066,6 @@ do { doubleget_union _tmp; \
10661066

10671067
#ifdef _WIN32
10681068
#define SO_EXT ".dll"
1069-
#elif defined(__APPLE__)
1070-
#define SO_EXT ".dylib"
10711069
#else
10721070
#define SO_EXT ".so"
10731071
#endif

libmariadb/CMakeLists.txt

+11-6
Original file line numberDiff line numberDiff line change
@@ -319,10 +319,15 @@ CREATE_EXPORT_FILE(mariadbclient
319319
"${MARIADB_LIB_SYMBOLS}")
320320

321321

322-
# CREATE OBJECT LIBRARY
323-
ADD_LIBRARY(mariadb_obj OBJECT ${LIBMARIADB_SOURCES})
324-
IF(UNIX)
325-
SET_TARGET_PROPERTIES(mariadb_obj PROPERTIES COMPILE_FLAGS "${CMAKE_SHARED_LIBRARY_C_FLAGS}")
322+
IF(CMAKE_VERSION VERSION_GREATER 2.8.7)
323+
# CREATE OBJECT LIBRARY
324+
ADD_LIBRARY(mariadb_obj OBJECT ${LIBMARIADB_SOURCES})
325+
IF(UNIX)
326+
SET_TARGET_PROPERTIES(mariadb_obj PROPERTIES COMPILE_FLAGS "${CMAKE_SHARED_LIBRARY_C_FLAGS}")
327+
ENDIF()
328+
SET (MARIADB_OBJECTS $<TARGET_OBJECTS:mariadb_obj>)
329+
ELSE()
330+
SET (MARIADB_OBJECTS ${LIBMARIADB_SOURCES})
326331
ENDIF()
327332

328333
# Xcode doesn't support targets that have only object files,
@@ -345,9 +350,9 @@ IF(WIN32)
345350
"FILE_DESCRIPTION:Static lib for client/server communication")
346351
ENDIF()
347352

348-
ADD_LIBRARY(mariadbclient STATIC ${mariadbclient_RC} $<TARGET_OBJECTS:mariadb_obj> ${EMPTY_FILE} mariadbclient.def)
353+
ADD_LIBRARY(mariadbclient STATIC ${mariadbclient_RC} ${MARIADB_OBJECTS} ${EMPTY_FILE} mariadbclient.def)
349354
TARGET_LINK_LIBRARIES(mariadbclient ${SYSTEM_LIBS})
350-
ADD_LIBRARY(libmariadb SHARED ${libmariadb_RC} $<TARGET_OBJECTS:mariadb_obj> ${EMPTY_FILE} mariadbclient.def)
355+
ADD_LIBRARY(libmariadb SHARED ${libmariadb_RC} ${MARIADB_OBJECTS} ${EMPTY_FILE} mariadbclient.def)
351356
TARGET_LINK_LIBRARIES(libmariadb ${SYSTEM_LIBS})
352357
IF(UNIX)
353358
SET_TARGET_PROPERTIES(libmariadb PROPERTIES COMPILE_FLAGS "${CMAKE_SHARED_LIBRARY_C_FLAGS}")

unittest/libmariadb/CMakeLists.txt

+4
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,10 @@
1313
# along with this program; if not, write to the Free Software
1414
# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
1515

16+
IF(SKIP_TESTS)
17+
RETURN()
18+
ENDIF()
19+
1620
ENABLE_TESTING()
1721

1822

0 commit comments

Comments
 (0)