Skip to content

chore: improve Makefile #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Nov 26, 2025
Merged

chore: improve Makefile #12

Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d507c27
chore: improve Makefile for minimal POSIX shell compatibility
mnencia Nov 26, 2025
a4ff35c
fix: dir variable expansion
NiccoloFei Nov 26, 2025
68c01ce
chore: do not spawn sub-makes
mnencia Nov 26, 2025
0f94425
chore: decouple push from all
mnencia Nov 26, 2025
bb2614d
chore: add a sanity check for three structure
mnencia Nov 26, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 8 additions & 10 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,13 @@
.SUFFIXES:

# Use bash if available, otherwise fall back to default shell
SHELL := $(shell which bash 2>/dev/null || echo /bin/sh)

# Find all directories containing metadata.hcl
FILES := $(shell find . -type f -name metadata.hcl)
DIRS := $(patsubst %/,%,$(patsubst ./%,%,$(dir $(FILES))))

.PHONY: all clean check prereqs $(DIRS)
.PHONY: all check prereqs push $(DIRS)

# Colours
GREEN := \033[0;32m
Expand All @@ -15,6 +18,8 @@ NC := \033[0m
# Dry run flag
DRY_RUN ?= false

default: all

# --------------------------
# Prerequisite checks
# --------------------------
Expand All @@ -32,7 +37,7 @@ prereqs:
check: prereqs
@echo -e "$(BLUE)Performing bake --check for all projects...$(NC)"
@$(foreach dir,$(DIRS), \
echo -e "$(BLUE)[CHECK] $dir$(NC)"; \
echo -e "$(BLUE)[CHECK] $(dir) $(NC)"; \
docker buildx bake -f $(dir)/metadata.hcl -f docker-bake.hcl --check; \
)

Expand All @@ -41,14 +46,7 @@ check: prereqs
# --------------------------
push: all
@echo -e "$(BLUE)Performing bake --push for all projects...$(NC)"
@$(foreach dir,$(DIRS), \
echo -e "$(BLUE)[PUSH] $dir$(NC)"; \
if [ "$(DRY_RUN)" = "true" ]; then \
echo -e "$(GREEN)[DRY RUN] docker buildx bake -f $(dir)/metadata.hcl -f docker-bake.hcl --push$(NC)"; \
else \
docker buildx bake -f $(dir)/metadata.hcl -f docker-bake.hcl --push; \
fi; \
)
@$(foreach dir,$(DIRS), $(MAKE) push-$(dir) DRY_RUN=$(DRY_RUN);)

# --------------------------
# Generic per-project push
Expand Down