如何遍历目录？Makefile循环执行任务报错求助

Hey there, let's get that Makefile sorted out! The syntax error you're seeing happens because of how Make interprets shell commands and line breaks—here's what's going wrong and how to fix it.

Why the Error Occurs #

Your original all-build command likely looks something like this (based on the error message):

all-build:
	for BIN in `ls cmd`; do
		# Your task command here

This breaks for two key reasons:

1. Make parses $ and backticks first: The backticks around ls cmd get interpreted by Make instead of being passed to the shell, leading to malformed command input.
2. Missing line continuation and closure: In Makefile shell commands, you need to end each line with a backslash (\) to tell Make the command continues on the next line, and you must close the for loop with done.

Fix 1: Correct the Shell Loop Syntax #

Update your all-build target to properly escape shell variables and handle line breaks:

all-build:
	@for BIN in $$(ls cmd); do \
		echo "Executing task for directory: $$BIN"; \
		# Replace this with your actual task command, e.g.: \
		# make build BIN=$$BIN; \
	done

• Use $$ instead of $ for shell variables (Make uses $ for its own variables, so we escape it with an extra $).
• Add \ at the end of every line in the loop to continue the command.
• Make sure to close the loop with done.

Fix 2: Use Make's Native Functions (More Reliable) #

Avoid relying on ls (which can break with special characters in directory names) and use Make's built-in functions instead. This is the cleaner, more robust approach:

# Get all directory names in the cmd/ folder
BINS := $(notdir $(wildcard cmd/*))

# Define the all-build target to depend on individual build targets
all-build: $(addsuffix -build, $(BINS))

# Generic target for each directory's build task
%-build:
	@echo "Running task for directory: $*"
	# Insert your task command here, using $* to reference the directory name
	# Example: go build -o bin/$* cmd/$*/main.go

This approach leverages Make's dependency system:

• $(wildcard cmd/*) gets all paths in cmd/
• $(notdir ...) strips the path to get just the directory names
• $(addsuffix -build, $(BINS)) creates targets like foo-build, bar-build for each directory
• The %-build pattern rule runs your task for each directory, with $* representing the directory name.

Which Fix to Choose? #

• Use Fix 1 if you need the flexibility of shell scripting (e.g., conditional logic in the loop).
• Use Fix 2 for a more idiomatic Makefile that integrates with Make's core features (better parallelism, dependency tracking).

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