Skip to content

Commit c1cf011

Browse files
authored
Subtle fix for ament_libraries_deduplicate tests (#516)
Evidently 'release' isn't a config keyword, but 'general' is. I also modified the test to assert that the keyword is actually treated like a keyword. Signed-off-by: Scott K Logan <[email protected]>
1 parent 34699da commit c1cf011

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

ament_cmake_libraries/test/test_deduplicate.cmake

+2-2
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,6 @@ ament_libraries_deduplicate(ACTUAL ${TEST_IN})
2626
assert_equal("debug;foo;debug;bar;debug;baz;bar" "${ACTUAL}")
2727

2828
# With mismatched build configs
29-
set(TEST_IN "debug;foo;debug;bar;debug;baz;release;bar")
29+
set(TEST_IN "optimized;foo;optimized;bar;general;baz;general;bar")
3030
ament_libraries_deduplicate(ACTUAL ${TEST_IN})
31-
assert_equal("debug;foo;debug;bar;debug;baz;release;bar" "${ACTUAL}")
31+
assert_equal("optimized;foo;optimized;bar;general;baz;general;bar" "${ACTUAL}")

0 commit comments

Comments
 (0)