Skip to content

Commit 9f4caeb

Browse files
committed
makefile: variables in make issue are now sorted
also, "\n get_variables" are removed from ISSEU_VARIABLES_NAMES. These two superfluous words were there previously, don't know why or how, but not easily spotted in an unsorted list. I ran the command below to test before/after + we have CI tests for make issue: make print-ISSUE_VARIABLES_NAMES Signed-off-by: Øyvind Harboe <[email protected]>
1 parent 53873f6 commit 9f4caeb

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

flow/util/utils.mk

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ $(foreach V, $(.VARIABLES),$(if $(filter-out $(1), $(origin $V)), $(if $(filter-
9090
endef
9191

9292
export UNSET_VARIABLES_NAMES := $(call get_variables,command% line environment% default automatic)
93-
export ISSUE_VARIABLES_NAMES := $(call get_variables,environment% default automatic)
93+
export ISSUE_VARIABLES_NAMES := $(sort $(filter-out \n get_variables, $(call get_variables,environment% default automatic)))
9494
export ISSUE_VARIABLES := $(foreach V, $(ISSUE_VARIABLES_NAMES), $(if $($V),$V=$($V),$V='')${\n})
9595
export COMMAND_LINE_ARGS := $(foreach V,$(.VARIABLES),$(if $(filter command% line, $(origin $V)),$(V)))
9696

0 commit comments

Comments
 (0)