fix(ci): do not flush the core when building standalone CHD tools

The CHD fixture step invoked make without TEST_EXPORTS after make test,
so the ABI stamp deleted the library and later host jobs had no .so.
Also exclude vendored libchdr from EditorConfig.

Co-authored-by: Cursor <[email protected]>
This commit is contained in:
Joseph MattielloandCursor authored and Joe Mattiello committed 2026-08-16 01:29:50 -04:00
1 parent 6095457b55
commit 84224695bd
2 files changed
+18 -1

No files matched your search

+9
View File
@@ -54,3 +54,12 @@ insert_final_newline = unset
trim_trailing_whitespace = unset
indent_style = unset
indent_size = unset
# Vendored libchdr (and its lzma/miniz/zstd deps). Do not reformat upstream.
[deps/libchdr/**]
charset = unset
end_of_line = unset
insert_final_newline = unset
trim_trailing_whitespace = unset
indent_style = unset
indent_size = unset
+9 -1
View File
@@ -877,7 +877,15 @@ else
MAKEFLAGS_LETTERS := $(filter-out -%,$(firstword $(MAKEFLAGS)))
DRY_RUN := $(strip $(findstring n,$(MAKEFLAGS_LETTERS)) \
$(filter -n --dry-run --just-print --recon,$(MAKEFLAGS)))
$(if $(DRY_RUN),,\
# Host-only libchdr binaries compile unity.c into the tool; they do not
# consume $(TARGET)/$(OBJECTS). A CI step that builds only these after
# `make test` (TEST_EXPORTS=1) must not treat the missing flag as an ABI
# flip and delete the core — that left Linux/macOS with no .so/.dylib
# for the later memory-map / instruction-set jobs.
STANDALONE_CHD_GOALS := tools/jagcd/jagcd-chd-check test/test_chd_unit
SKIP_BUILD_FLUSH := $(if $(MAKECMDGOALS),\
$(if $(filter-out $(STANDALONE_CHD_GOALS),$(MAKECMDGOALS)),,1))
$(if $(or $(DRY_RUN),$(SKIP_BUILD_FLUSH)),,\
$(shell [ "$$(cat $(BUILD_CONFIG_STAMP) 2>/dev/null)" = "$(BUILD_CONFIG)" ] \
|| { printf '%s' "$(BUILD_CONFIG)" > $(BUILD_CONFIG_STAMP); \
rm -f $(TARGET) $(OBJECTS) $(LEGACY_LINK_MODE_STAMP); }))