Makefile变量默认值不生效？传入空值触发错误的原因排查

你的问题核心出在Makefile变量的优先级规则，以及对“未传入变量”和“传入空值变量”两种场景的判断逻辑混淆上：

1. 命令行变量优先级高于Makefile内定义的变量
   当你执行make BRANCH=或make BRANCH=""时，已经通过命令行显式定义了BRANCH变量（值为空字符串）。而Makefile的规则是：命令行传入的变量优先级远高于Makefile内部用=或:=定义的变量。所以即使开头的ifeq ($(BRANCH), )条件成立（因为值为空），后续的BRANCH := $(shell ...)也无法覆盖命令行传入的空值，最终BRANCH还是保持为空。

2. 空值判断的两种场景差异

• 当你不传入BRANCH时，变量处于未定义状态，$(BRANCH)展开为空字符串，开头的ifeq条件成立，会执行默认值设置，此时BRANCH被赋值为当前分支，后续目标内的判断自然不会触发错误。
• 当你传入BRANCH=时，变量是已定义但值为空，开头的默认值设置因为优先级问题无法生效，到目标all执行时，ifeq ($(BRANCH), )条件依然成立，所以触发错误提示。

根据你的需求（未传入则设默认，传入空值时可选处理），有两种调整方式：

方式1：允许传入空值时自动使用默认值 #

如果希望用户即使误传空值，也自动回退到默认值，需要用override关键字强制覆盖命令行变量：

ifeq ($(BRANCH), )
    override BRANCH := $(shell git rev-parse --abbrev-ref HEAD)
endif
ifeq ($(COMMIT), )
    # 注：原代码中COMMIT默认值用了分支名的命令，这里修正为获取提交哈希的正确命令
    override COMMIT := $(shell git rev-parse HEAD)
endif

all:
ifeq ($(BRANCH), )
    $(error Please pass a valid branch)
endif
ifeq ($(COMMIT), )
    $(error Please pass a valid commit)
endif
    echo $(BRANCH) $(COMMIT)
    rm -f server-test

方式2：明确禁止传入空值（保持现有报错逻辑） #

如果你的需求就是不允许用户传入空值，那当前逻辑是符合预期的——开头的默认值只处理“未传入变量”的场景，传入空值时就触发错误。此时只需把COMMIT的默认值修正为正确的提交哈希即可：

ifeq ($(BRANCH), )
    BRANCH := $(shell git rev-parse --abbrev-ref HEAD)
endif
ifeq ($(COMMIT), )
    COMMIT := $(shell git rev-parse HEAD)
endif

all:
ifeq ($(BRANCH), )
    $(error Please pass a valid branch)
endif
ifeq ($(COMMIT), )
    $(error Please pass a valid commit)
endif
    echo $(BRANCH) $(COMMIT)
    rm -f server-test

内容的提问来源于stack exchange，提问作者VinayakaB