This is an automated email from the ASF dual-hosted git repository.

leandron pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git


The following commit(s) were added to refs/heads/main by this push:
     new 977bdbd  Force CMake targets in top-level Makefile to run (#8840)
977bdbd is described below

commit 977bdbdf772f6149554d43ae9073ce58c8e36a38
Author: Christopher Sidebottom <[email protected]>
AuthorDate: Wed Aug 25 16:54:40 2021 +0100

    Force CMake targets in top-level Makefile to run (#8840)
    
    This is a bug I introduced in https://github.com/apache/tvm/pull/8809, 
because the built binary is now named `build/cpptest` when `make` checks that 
artifact it finds it exists already and skips running `make -C build cpptest`. 
This ensures all nested `make` calls are forced to run from the top-level 
`Makefile`.
---
 Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 58cb1b6..8ebbea6 100644
--- a/Makefile
+++ b/Makefile
@@ -81,7 +81,7 @@ FORCE:
 CMAKE_TARGETS = all runtime vta cpptest crttest
 
 define GEN_CMAKE_RULE
-%/$(CMAKE_TARGET): %/CMakeCache.txt
+%/$(CMAKE_TARGET): %/CMakeCache.txt FORCE
        @$$(MAKE) -C $$(@D) $(CMAKE_TARGET)
 endef
 $(foreach CMAKE_TARGET,$(CMAKE_TARGETS),$(eval $(GEN_CMAKE_RULE)))

Reply via email to