182

如何根据未设置/赋值的makefile变量中止make/makefile执行?

我想出了这个,但只有在调用者没有明确运行目标(即make只运行)时才有效。

ifeq ($(MY_FLAG),)
abort:   ## This MUST be the first target :( ugly
    @echo Variable MY_FLAG not set && false
endif

all:
    @echo MY_FLAG=$(MY_FLAG)

我认为这样的事情是个好主意,但在make的手册中没有找到任何东西:

ifndef MY_FLAG
.ABORT
endif
4

6 回答 6

343

TL;DR:使用error功能

ifndef MY_FLAG
$(error MY_FLAG is not set)
endif

请注意,这些行不能缩进。更准确地说,这些行之前不能有制表符。


通用解决方案

如果您要测试许多变量,则值得为此定义一个辅助函数:

# Check that given variables are set and all have non-empty values,
# die with an error otherwise.
#
# Params:
#   1. Variable name(s) to test.
#   2. (optional) Error message to print.
check_defined = \
    $(strip $(foreach 1,$1, \
        $(call __check_defined,$1,$(strip $(value 2)))))
__check_defined = \
    $(if $(value $1),, \
      $(error Undefined $1$(if $2, ($2))))

以下是如何使用它:

$(call check_defined, MY_FLAG)

$(call check_defined, OUT_DIR, build directory)
$(call check_defined, BIN_DIR, where to put binary artifacts)
$(call check_defined, \
            LIB_INCLUDE_DIR \
            LIB_SOURCE_DIR, \
        library path)


这将输出如下错误:

Makefile:17: *** Undefined OUT_DIR (build directory).  Stop.

笔记:

真正的检查在这里完成:

$(if $(value $1),,$(error ...))

这反映了ifndef条件的行为,因此定义为空值的变量也被认为是“未定义的”。但这仅适用于简单变量和显式空递归变量:

# ifndef and check_defined consider these UNDEFINED:
explicitly_empty =
simple_empty := $(explicitly_empty)

# ifndef and check_defined consider it OK (defined):
recursive_empty = $(explicitly_empty)

正如@VictorSergienko 在评论中所建议的那样,可能需要稍微不同的行为:

$(if $(value $1)测试该值是否为非空。如果变量定义为空值,有时也可以。我会用$(if $(filter undefined,$(origin $1)) ...

和:

此外,如果它是一个目录并且在运行检查时它必须存在,我会使用$(if $(wildcard $1)). 但将是另一个功能。

特定目标检查

还可以扩展解决方案,以便仅在调用某个目标时才需要变量。

$(call check_defined, ...)从食谱里面

只需将支票移入配方:

foo :
    @:$(call check_defined, BAR, baz value)

前导@符号关闭命令回显,并且:是实际命令,即 shell no-op stub

显示目标名称

check_defined功能可以改进为还输出目标名称(通过$@变量提供):

check_defined = \
    $(strip $(foreach 1,$1, \
        $(call __check_defined,$1,$(strip $(value 2)))))
__check_defined = \
    $(if $(value $1),, \
        $(error Undefined $1$(if $2, ($2))$(if $(value @), \
                required by target `$@')))

因此,现在失败的检查会产生格式良好的输出:

Makefile:7: *** Undefined BAR (baz value) required by target `foo'.  Stop.

check-defined-MY_FLAG特殊目标

我个人会使用上面简单直接的解决方案。但是,例如,此答案建议使用特殊目标来执行实际检查。可以尝试将其概括并将目标定义为隐式模式规则:

# Check that a variable specified through the stem is defined and has
# a non-empty value, die with an error otherwise.
#
#   %: The name of the variable to test.
#   
check-defined-% : __check_defined_FORCE
    @:$(call check_defined, $*, target-specific)

# Since pattern rules can't be listed as prerequisites of .PHONY,
# we use the old-school and hackish FORCE workaround.
# You could go without this, but otherwise a check can be missed
# in case a file named like `check-defined-...` exists in the root 
# directory, e.g. left by an accidental `make -t` invocation.
.PHONY : __check_defined_FORCE
__check_defined_FORCE :

用法:

foo :|check-defined-BAR

请注意,check-defined-BAR被列为仅订单( |...) 先决条件。

优点:

  • (可以说)更简洁的语法

缺点:

eval我相信,可以使用一些魔法和二次扩展技巧来克服这些限制,尽管我不确定这是否值得。

于 2012-06-01T22:57:53.963 回答
47

使用外壳功能test

foo:
    test $(something)

用法:

$ make foo
test 
Makefile:2: recipe for target 'foo' failed
make: *** [foo] Error 1
$ make foo something=x
test x
于 2017-01-18T14:35:57.010 回答
16

您可以使用 IF 来测试:

check:
        @[ "${var}" ] || ( echo ">> var is not set"; exit 1 )

结果:

$ make check
>> var is not set
Makefile:2: recipe for target 'check' failed
make: *** [check] Error 1
于 2018-07-09T14:29:02.007 回答
7

对未设置的变量使用 shell 错误处理(注意 double $):

$ cat Makefile
foo:
        echo "something is set to $${something:?}"

$ make foo
echo "something is set to ${something:?}"
/bin/sh: something: parameter null or not set
make: *** [foo] Error 127


$ make foo something=x
echo "something is set to ${something:?}"
something is set to x

如果您需要自定义错误消息,请将其添加到?

$ cat Makefile
hello:
        echo "hello $${name:?please tell me who you are via \$$name}"

$ make hello
echo "hello ${name:?please tell me who you are via \$name}"
/bin/sh: name: please tell me who you are via $name
make: *** [hello] Error 127

$ make hello name=jesus
echo "hello ${name:?please tell me who you are via \$name}"
hello jesus
于 2018-06-01T11:29:31.300 回答
5

为简单起见:

$ cat Makefile
check-%:
        @: $(if $(value $*),,$(error $* is undefined))

bar:| check-foo
        echo "foo is $$foo"

输出:

$ make bar
Makefile:2: *** foo is undefined. Stop.
$ make bar foo="something"
echo "foo is $$foo"
foo is something
于 2020-03-07T11:44:36.527 回答
0

另外的选择:

MY_FLAG = $(error Please set this flag)

尝试在任何地方使用此变量都会导致错误,除非它被命令行覆盖。

要接受环境变量,请使用?=

MY_FLAG ?= $(error Please set this flag)
于 2022-02-23T19:49:03.990 回答