Skip to content

Commit cf002e9

Browse files
committed
Tools should not be build prereqs of tests. Closes #3946
1 parent 3e850c3 commit cf002e9

File tree

4 files changed

+23
-14
lines changed

4 files changed

+23
-14
lines changed

Makefile.in

Lines changed: 18 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -341,6 +341,7 @@ HSREQ$(1)_H_$(3) = \
341341
$$(HLIB$(1)_H_$(3))/$$(CFG_RUSTLLVM) \
342342
$$(HCORELIB_DEFAULT$(1)_H_$(3)) \
343343
$$(HSTDLIB_DEFAULT$(1)_H_$(3)) \
344+
$$(HLIBSYNTAX_DEFAULT$(1)_H_$(3)) \
344345
$$(HLIBRUSTC_DEFAULT$(1)_H_$(3)) \
345346
$$(MKFILE_DEPS)
346347

@@ -350,17 +351,31 @@ TSREQ$(1)_T_$(2)_H_$(3) = \
350351
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_RUNTIME) \
351352
$$(TLIB$(1)_T_$(2)_H_$(3))/libmorestack.a
352353

353-
# Prerequisites for complete stageN targets
354+
# Prerequisites for a working stageN compiler and libraries
354355
SREQ$(1)_T_$(2)_H_$(3) = \
355356
$$(TSREQ$(1)_T_$(2)_H_$(3)) \
356357
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_CORELIB) \
358+
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_STDLIB)
359+
360+
# Prerequisites for a working stageN compiler and libraries
361+
CSREQ$(1)_T_$(2)_H_$(3) = \
362+
$$(TSREQ$(1)_T_$(2)_H_$(3)) \
363+
$$(HBIN$(1)_H_$(3))/fuzzer$$(X) \
364+
$$(HBIN$(1)_H_$(3))/cargo$$(X) \
365+
$$(HBIN$(1)_H_$(3))/rustdoc$$(X) \
366+
$$(HBIN$(1)_H_$(3))/rusti$$(X) \
367+
$$(HLIB$(1)_H_$(3))/$$(CFG_LIBFUZZER) \
368+
$$(HLIB$(1)_H_$(3))/$$(CFG_LIBCARGO) \
369+
$$(HLIB$(1)_H_$(3))/$$(CFG_LIBRUSTDOC) \
370+
$$(HLIB$(1)_H_$(3))/$$(CFG_LIBRUSTI) \
371+
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_CORELIB) \
357372
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_STDLIB) \
358373
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_LIBSYNTAX) \
359374
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_LIBRUSTC) \
360375
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_LIBFUZZER) \
361376
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_LIBCARGO) \
362377
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_LIBRUSTDOC) \
363-
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_LIBRUSTI) \
378+
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_LIBRUSTI)
364379

365380
ifeq ($(1),0)
366381
# Don't run the the stage0 compiler under valgrind - that ship has sailed
@@ -461,15 +476,7 @@ all: $(SREQ1$(CFG_HOST_TRIPLE)) $(GENERATED) docs
461476

462477
else
463478

464-
TSREQS := \
465-
$(foreach target,$(CFG_TARGET_TRIPLES), \
466-
$(SREQ3_T_$(target)_H_$(CFG_HOST_TRIPLE)))
467-
FUZZ := $(HBIN2_H_$(CFG_HOST_TRIPLE))/fuzzer$(X)
468-
CARGO := $(HBIN2_H_$(CFG_HOST_TRIPLE))/cargo$(X)
469-
RUSTDOC := $(HBIN2_H_$(CFG_HOST_TRIPLE))/rustdoc$(X)
470-
RUSTI := $(HBIN2_H_$(CFG_HOST_TRIPLE))/rusti$(X)
471-
472-
all: rustc $(GENERATED) docs $(FUZZ) $(CARGO) $(RUSTDOC) $(RUSTI)
479+
all: $(CSREQ3$(CFG_HOST_TRIPLE)) $(GENERATED) docs
473480

474481
endif
475482

mk/install.mk

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ PTR$(1)$(2) = $$(PREFIX_LIB)/rustc/$(1)
3333
PTB$(1)$(2) = $$(PTR$(1)$(2))/bin
3434
PTL$(1)$(2) = $$(PTR$(1)$(2))/$(CFG_LIBDIR)
3535

36-
install-target-$(1)-host-$(2): $$(SREQ$$(ISTAGE)_T_$(1)_H_$(2))
36+
install-target-$(1)-host-$(2): $$(CSREQ$$(ISTAGE)_T_$(1)_H_$(2))
3737
$$(Q)mkdir -p $$(PTL$(1)$(2))
3838
$$(Q)$$(call INSTALL,$$(TL$(1)$(2)),$$(PTL$(1)$(2)),$$(CFG_RUNTIME))
3939
$$(Q)$$(call INSTALL_LIB, \
@@ -74,7 +74,7 @@ PHB = $(PREFIX_BIN)
7474
# Shorthand for the prefix bin directory
7575
PHL = $(PREFIX_LIB)
7676

77-
install-host: $(SREQ$(ISTAGE)_T_$(CFG_HOST_TRIPLE)_H_$(CFG_HOST_TRIPLE))
77+
install-host: $(CSREQ$(ISTAGE)_T_$(CFG_HOST_TRIPLE)_H_$(CFG_HOST_TRIPLE))
7878
$(Q)mkdir -p $(PREFIX_BIN)
7979
$(Q)mkdir -p $(PREFIX_LIB)
8080
$(Q)mkdir -p $(PREFIX_ROOT)/share/man/man1

mk/tests.mk

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -352,7 +352,8 @@ $(3)/test/rustctest.stage$(1)-$(2)$$(X): \
352352
$$(COMPILER_CRATE) \
353353
$$(COMPILER_INPUTS) \
354354
$$(SREQ$(1)_T_$(2)_H_$(3)) \
355-
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_RUSTLLVM)
355+
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_RUSTLLVM) \
356+
$$(TLIB$(1)_T_$(2)_H_$(3))/$$(CFG_LIBSYNTAX)
356357
@$$(call E, compile_and_link: $$@)
357358
$$(STAGE$(1)_T_$(2)_H_$(3)) -o $$@ $$< --test
358359

src/test/compile-fail/qquote-1.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
// xfail-test Can't use syntax crate here
12

23
extern mod std;
34
extern mod syntax;

0 commit comments

Comments
 (0)