Skip to content

Commit 2b7eee4

Browse files
committed
8366899: SetupExecute should add the command line to vardeps
Reviewed-by: erikj
1 parent 87d5042 commit 2b7eee4

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

make/common/Execute.gmk

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -148,9 +148,12 @@ define SetupExecuteBody
148148
$1_INFO := Running commands for $1
149149
endif
150150

151+
$1_VARDEPS := $$($1_COMMAND) $$($1_PRE_COMMAND) $$($1_POST_COMMAND)
152+
$1_VARDEPS_FILE := $$(call DependOnVariable, $1_VARDEPS)
153+
151154
ifneq ($$($1_PRE_COMMAND), )
152155

153-
$$($1_PRE_MARKER): $$($1_DEPS)
156+
$$($1_PRE_MARKER): $$($1_DEPS) $$($1_VARDEPS_FILE)
154157
ifneq ($$($1_WARN), )
155158
$$(call LogWarn, $$($1_WARN))
156159
endif
@@ -176,7 +179,7 @@ define SetupExecuteBody
176179

177180
$1 := $$($1_PRE_MARKER) $$($1_EXEC_RESULT)
178181
else
179-
$$($1_EXEC_RESULT): $$($1_DEPS)
182+
$$($1_EXEC_RESULT): $$($1_DEPS) $$($1_VARDEPS_FILE)
180183
ifneq ($$($1_WARN), )
181184
$$(call LogWarn, $$($1_WARN))
182185
endif

0 commit comments

Comments
 (0)