This fixes an error in the Makefile-dependency of unittests, introduced
in commit ad40427a864a7436953b319fbfdb2c04ccb7bdab.

>From the perspective of Waffle's CMake, libcmocka.so is an *imported*
library. In other words, it's a virtual target, represented by
a '.PHONY' target in Unix Makefiles.

To guarantee that libcmocka.so gets built, CMake must declare that the
.PHONY 'cmocka' target depends on another .PHONY target, which does the
actual building. Making 'cmocka' depend on a non-PHONY target doesn't
work.

Reported-by: Shazia Rahman <[email protected]>
Reported-by: Paul Berry <[email protected]>
Signed-off-by: Chad Versace <[email protected]>
---
 cmake/Modules/WaffleCMocka.cmake | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cmake/Modules/WaffleCMocka.cmake b/cmake/Modules/WaffleCMocka.cmake
index 635c31a..e276acd 100644
--- a/cmake/Modules/WaffleCMocka.cmake
+++ b/cmake/Modules/WaffleCMocka.cmake
@@ -43,7 +43,7 @@ set_target_properties(cmocka
     PROPERTIES
         IMPORTED_LOCATION ${cmocka_library_location}
 )
-add_dependencies(cmocka ${cmocka_library_location})
+add_dependencies(cmocka cmocka-build)
 add_custom_command(
     OUTPUT ${cmocka_library_location}
     WORKING_DIRECTORY ${cmocka_build_dir}
-- 
1.8.3.1

_______________________________________________
waffle mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/waffle

Reply via email to