Skip to content

Commit 94a19f1

Browse files
committed
windows/Makefile: Update test dependency on $(PROG).
PR #9012 (b2e8240) changed the output to $(BUILD)/$(PROG) but the tests are still looking for $(PROG). Also remove the now-unnecessary override of $(PROG) in the standard variant. Signed-off-by: Jim Mussared <jim.mussared@gmail.com>
1 parent bfc63a5 commit 94a19f1

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

ports/windows/Makefile

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ include $(VARIANT_DIR)/mpconfigvariant.mk
1616
FROZEN_MANIFEST ?= variants/manifest.py
1717

1818
# Define main target
19-
# This should be configured by the mpconfigvariant.mk
2019
PROG ?= micropython
2120

2221
# qstr definitions (must come before including py.mk)
@@ -98,10 +97,10 @@ include $(TOP)/py/mkrules.mk
9897

9998
RUN_TESTS_SKIP += -e math_fun -e float2int_double -e float_parse -e math_domain_special
10099

101-
test: $(PROG) $(TOP)/tests/run-tests.py
100+
test: $(BUILD)/$(PROG) $(TOP)/tests/run-tests.py
102101
$(eval DIRNAME=ports/$(notdir $(CURDIR)))
103-
cd $(TOP)/tests && MICROPY_MICROPYTHON=../$(DIRNAME)/$(PROG) $(PYTHON) ./run-tests.py $(RUN_TESTS_SKIP)
102+
cd $(TOP)/tests && MICROPY_MICROPYTHON=../$(DIRNAME)/$(BUILD)/$(PROG) $(PYTHON) ./run-tests.py $(RUN_TESTS_SKIP)
104103

105104
test_full: test
106105
$(eval DIRNAME=ports/$(notdir $(CURDIR)))
107-
cd $(TOP)/tests && MICROPY_MICROPYTHON=../$(DIRNAME)/$(PROG) $(PYTHON) ./run-tests.py --via-mpy $(RUN_TESTS_MPY_CROSS_FLAGS) $(RUN_TESTS_SKIP) -d basics float micropython
106+
cd $(TOP)/tests && MICROPY_MICROPYTHON=../$(DIRNAME)/$(BUILD)/$(PROG) $(PYTHON) ./run-tests.py --via-mpy $(RUN_TESTS_MPY_CROSS_FLAGS) $(RUN_TESTS_SKIP) -d basics float micropython
Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1 @@
11
# This is the default variant when you `make` the Windows port.
2-
3-
PROG ?= micropython

0 commit comments

Comments
 (0)