Skip to content

Commit

Permalink
Teach the makefile to use multiple src-base's
Browse files Browse the repository at this point in the history
  • Loading branch information
emberian committed Aug 28, 2013
1 parent 8a07f57 commit 43f851d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
5 changes: 3 additions & 2 deletions mk/tests.mk
Original file line number Diff line number Diff line change
Expand Up @@ -516,7 +516,7 @@ CTEST_BUILD_BASE_cfail = compile-fail
CTEST_MODE_cfail = compile-fail
CTEST_RUNTOOL_cfail = $(CTEST_RUNTOOL)

CTEST_SRC_BASE_bench = bench
CTEST_SRC_BASE_bench = bench bench/rt bench/shootout bench/std
CTEST_BUILD_BASE_bench = bench
CTEST_MODE_bench = run-pass
CTEST_RUNTOOL_bench = $(CTEST_RUNTOOL)
Expand Down Expand Up @@ -610,7 +610,8 @@ define DEF_RUN_COMPILETEST

CTEST_ARGS$(1)-T-$(2)-H-$(3)-$(4) := \
$$(CTEST_COMMON_ARGS$(1)-T-$(2)-H-$(3)) \
--src-base $$(S)src/test/$$(CTEST_SRC_BASE_$(4))/ \
$(foreach base,$$(CTEST_SRC_BASE_$(4)), \
--src-base $$(S)src/test/$$(base))/ \
--build-base $(3)/test/$$(CTEST_BUILD_BASE_$(4))/ \
--ratchet-metrics $(call TEST_RATCHET_FILE,$(1),$(2),$(3),$(4)) \
--mode $$(CTEST_MODE_$(4)) \
Expand Down
2 changes: 1 addition & 1 deletion src/compiletest/compiletest.rs
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ pub fn parse_config(args: ~[~str]) -> config {
fn opt_path(m: &getopts::Matches, nm: &str) -> Path {
Path(getopts::opt_str(m, nm))
}

let src_base = getopts::opt_strs(matches, "src-base");

config {
Expand Down

13 comments on commit 43f851d

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 28, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 28, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging cmr/rust/test-restructure = 43f851d into auto

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 28, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cmr/rust/test-restructure = 43f851d merged ok, testing candidate = dc3296af

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 28, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 29, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 29, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging cmr/rust/test-restructure = 43f851d into auto

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 29, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cmr/rust/test-restructure = 43f851d merged ok, testing candidate = 800e75b2

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 29, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 30, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 30, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging cmr/rust/test-restructure = 43f851d into auto

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 30, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cmr/rust/test-restructure = 43f851d merged ok, testing candidate = 7c6c751

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 30, 2013

@bors
Copy link
Contributor

@bors bors commented on 43f851d Aug 30, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 7c6c751

Please sign in to comment.