Skip to content

Commit

Permalink
Fix .testing make for regression tests
Browse files Browse the repository at this point in the history
This commit fixes a bug that target code cannot be built for regression
tests when BUILD is specified with command line arguments. The fix
explicitly specifies BUILD argument for target codebase, avoiding the
variable being passed from the parent make.
  • Loading branch information
herrwang0 authored and marshallward committed Oct 4, 2024
1 parent e24cd3a commit 5dcc8e0
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .testing/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -282,7 +282,7 @@ $(BUILD)/%/MOM6: $(BUILD)/%/Makefile FORCE

# Target codebase should use its own build system
$(BUILD)/target/MOM6: $(BUILD)/target FORCE | $(TARGET_CODEBASE)
$(MAKE) -C $(TARGET_CODEBASE)/.testing build/symmetric/MOM6
$(MAKE) -C $(TARGET_CODEBASE)/.testing BUILD=build build/symmetric/MOM6

$(BUILD)/target: | $(TARGET_CODEBASE)
ln -s $(abspath $(TARGET_CODEBASE))/.testing/build/symmetric $@
Expand Down

0 comments on commit 5dcc8e0

Please sign in to comment.