Skip to content

Commit 88ec9ee

Browse files
authored
[test][embedded] Declare embedded-synchronization as a test dependency (#72356)
PR #72293 added the synchronization module to embedded, and it was added as dependency of `ALL`, but not of the `-test-depends` targets. If one is using the `-test-depends` targets, the synchronization library for embedded will not be build, and the test will fail. This is a follow up to #72329. I had a partial build to test that, and I did not notice this was also missing.
1 parent b57db86 commit 88ec9ee

File tree

1 file changed

+13
-8
lines changed

1 file changed

+13
-8
lines changed

test/CMakeLists.txt

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -314,14 +314,19 @@ foreach(SDK ${SWIFT_SDKS})
314314
endif()
315315
endif()
316316

317-
if(SWIFT_SHOULD_BUILD_EMBEDDED_STDLIB AND TARGET "embedded-stdlib")
318-
list(APPEND test_dependencies "embedded-stdlib")
319-
endif()
320-
if(SWIFT_SHOULD_BUILD_EMBEDDED_STDLIB AND TARGET "embedded-darwin")
321-
list(APPEND test_dependencies "embedded-darwin")
322-
endif()
323-
if(SWIFT_SHOULD_BUILD_EMBEDDED_STDLIB AND TARGET "embedded-concurrency")
324-
list(APPEND test_dependencies "embedded-concurrency")
317+
if(SWIFT_SHOULD_BUILD_EMBEDDED_STDLIB)
318+
if(TARGET "embedded-stdlib")
319+
list(APPEND test_dependencies "embedded-stdlib")
320+
endif()
321+
if(TARGET "embedded-darwin")
322+
list(APPEND test_dependencies "embedded-darwin")
323+
endif()
324+
if(TARGET "embedded-concurrency")
325+
list(APPEND test_dependencies "embedded-concurrency")
326+
endif()
327+
if(TARGET "embedded-synchronization")
328+
list(APPEND test_dependencies "embedded-synchronization")
329+
endif()
325330
endif()
326331

327332
if(NOT "${COVERAGE_DB}" STREQUAL "")

0 commit comments

Comments
 (0)