Skip to content

Commit 51e5a15

Browse files
committed
Bug#22824957: MERGING OF INNODB AND KEYRING UNIT TESTS DOES NOT WORK PROPERLY
Merging of Innodb and Keyring unit tests did not work properly because ADD_SUBDIRECTORY for these two directories was done before the MERGE_UNITTESTS CMake option was declared. This mean that on these unit tests were not merged properly on the initial build. That meant individual build and linking for each unit tests - which takes time and disk space. This patch fixes the problem by moving ADD_SUBDIRECTORY after MERGE_UNITTESTS has been defined.
1 parent 87d8775 commit 51e5a15

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

unittest/gunit/CMakeLists.txt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -212,9 +212,6 @@ IF (WITH_SSL STREQUAL "bundled")
212212
ADD_SUBDIRECTORY(yassl)
213213
ENDIF()
214214

215-
ADD_SUBDIRECTORY(innodb)
216-
ADD_SUBDIRECTORY(keyring)
217-
218215
INCLUDE_DIRECTORIES(
219216
${CMAKE_SOURCE_DIR}/include
220217
${CMAKE_SOURCE_DIR}/libbinlogevents/include
@@ -491,3 +488,6 @@ ENDIF(MERGE_UNITTESTS)
491488

492489
## Most executables depend on libeay32.dll (through mysys_ssl).
493490
COPY_OPENSSL_DLLS(copy_openssl_gunit)
491+
492+
ADD_SUBDIRECTORY(innodb)
493+
ADD_SUBDIRECTORY(keyring)

0 commit comments

Comments
 (0)