Skip to content

Commit

Permalink
build: Generalise ".in" file rule
Browse files Browse the repository at this point in the history
Make the rule that generates the configuration file
more general to allow other files to be auto-generated.

Signed-off-by: James O. D. Hunt <[email protected]>
  • Loading branch information
jodh-intel committed Sep 25, 2017
1 parent fef6ba9 commit ffe6ccf
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,6 @@ COMMIT := $(if $(shell git status --porcelain --untracked-files=no),${COMMIT_NO}

CONFIG_FILE = configuration.toml
CONFIG = config/$(CONFIG_FILE)
CONFIG_IN = $(CONFIG).in

DESTTARGET := $(abspath $(DESTBINDIR)/$(TARGET))

Expand Down Expand Up @@ -249,12 +248,12 @@ endef
export GENERATED_CODE


GENERATED_FILES += config-generated.go
GENERATED_GO_FILES += config-generated.go

config-generated.go: Makefile VERSION
$(QUIET_GENERATE)echo "$$GENERATED_CODE" >$@

$(TARGET): $(EXTRA_DEPS) $(SOURCES) $(GENERATED_FILES) Makefile | show-summary
$(TARGET): $(EXTRA_DEPS) $(SOURCES) $(GENERATED_GO_FILES) $(GENERATED_FILES) Makefile | show-summary
$(QUIET_BUILD)go build -i -o $@ .

pause: pause/pause.go
Expand All @@ -276,8 +275,12 @@ pause: pause/pause.go
$(TARGET).coverage: $(SOURCES) $(GENERATED_FILES) Makefile
$(QUIET_TEST)go test -o $@ -covermode count

$(CONFIG): $(CONFIG_IN) $(GENERATED_FILES)
GENERATED_FILES += $(CONFIG)

$(GENERATED_FILES): %: %.in Makefile VERSION
$(QUIET_CONFIG)$(SED) \
-e "s|@COMMIT@|$(COMMIT)|g" \
-e "s|@VERSION@|$(VERSION)|g" \
-e "s|@CONFIG_IN@|$(CONFIG_IN)|g" \
-e "s|@IMAGEPATH@|$(IMAGEPATH)|g" \
-e "s|@KERNELPATH@|$(KERNELPATH)|g" \
Expand Down Expand Up @@ -326,7 +329,7 @@ install-scripts:
$(QUIET_INST)install --mode 0755 -D $(SCRIPTS) $(SCRIPTS_DIR)

clean:
$(QUIET_CLEAN)rm -f $(TARGET) $(CONFIG) $(GENERATED_FILES)
$(QUIET_CLEAN)rm -f $(TARGET) $(CONFIG) $(GENERATED_GO_FILES) $(GENERATED_FILES)
$(QUIET_CLEAN)rm -f pause/pause

show-usage: show-header
Expand Down

0 comments on commit ffe6ccf

Please sign in to comment.