Skip to content

Commit ff61d3c

Browse files
jnasbyupgradeclaude
andcommitted
Makefile: use .sql.tmp intermediate in test build rules
Write to $@.tmp first, then atomically mv to $@, so a failed pipeline never leaves a partial file that appears complete. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 7dd8fe5 commit ff61d3c

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

Makefile

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,10 +72,12 @@ $(TEST_BUILD_DIR)/active.sql: $(TEST_BUILD_DIR)/dep.mk $(TEST_BUILD_DIR)/$(TEST_
7272
ln -sf $(TEST_LOAD_SOURCE).sql $@
7373

7474
$(TEST_BUILD_DIR)/upgrade.sql: test/load_upgrade.sql $(TEST_BUILD_DIR) old_version
75-
(echo @generated@ && cat $< && echo @generated@) | sed -e 's#@generated@#-- GENERATED FILE! DO NOT EDIT! See $<#' > $@
75+
(echo @generated@ && cat $< && echo @generated@) | sed -e 's#@generated@#-- GENERATED FILE! DO NOT EDIT! See $<#' > $@.tmp
76+
mv $@.tmp $@
7677

7778
$(TEST_BUILD_DIR)/new.sql: test/load_new.sql $(TEST_BUILD_DIR)
78-
(echo @generated@ && cat $< && echo @generated@) | sed -e 's#@generated@#-- GENERATED FILE! DO NOT EDIT! See $<#' > $@
79+
(echo @generated@ && cat $< && echo @generated@) | sed -e 's#@generated@#-- GENERATED FILE! DO NOT EDIT! See $<#' > $@.tmp
80+
mv $@.tmp $@
7981

8082
# TODO: figure out vpath
8183
EXTRA_CLEAN += $(TEST_BUILD_DIR)/

0 commit comments

Comments
 (0)