Skip to content

Commit

Permalink
Deny warnings in stage1+ libsyntax/librustc/librustdoc/librusti/librust
Browse files Browse the repository at this point in the history
  • Loading branch information
@erickt
erickt committed Aug 17, 2013
1 parent ad5c676 commit cc56708
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions mk/target.mk
View file
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ $$(TLIB$(1)_T_$(2)_H_$(3))/$(CFG_LIBSYNTAX_$(3)): \
| $$(TLIB$(1)_T_$(2)_H_$(3))/
@$$(call E, compile_and_link: $$@)
$$(call REMOVE_ALL_OLD_GLOB_MATCHES_EXCEPT,$$(dir $$@),$(LIBSYNTAX_GLOB_$(2)),$$(notdir $$@))
$$(STAGE$(1)_T_$(2)_H_$(3)) $(BORROWCK) --out-dir $$(@D) $$< && touch $$@
$$(STAGE$(1)_T_$(2)_H_$(3)) $$(WFLAGS_ST$(1)) $(BORROWCK) --out-dir $$(@D) $$< && touch $$@
$$(call LIST_ALL_OLD_GLOB_MATCHES_EXCEPT,$$(dir $$@),$(LIBSYNTAX_GLOB_$(2)),$$(notdir $$@))

# Only build the compiler for host triples
Expand All @@ -90,7 +90,7 @@ $$(TLIB$(1)_T_$(2)_H_$(3))/$(CFG_LIBRUSTC_$(3)): \
| $$(TLIB$(1)_T_$(2)_H_$(3))/
@$$(call E, compile_and_link: $$@)
$$(call REMOVE_ALL_OLD_GLOB_MATCHES_EXCEPT,$$(dir $$@),$(LIBRUSTC_GLOB_$(2)),$$(notdir $$@))
$$(STAGE$(1)_T_$(2)_H_$(3)) --out-dir $$(@D) $$< && touch $$@
$$(STAGE$(1)_T_$(2)_H_$(3)) $$(WFLAGS_ST$(1)) --out-dir $$(@D) $$< && touch $$@
$$(call LIST_ALL_OLD_GLOB_MATCHES_EXCEPT,$$(dir $$@),$(LIBRUSTC_GLOB_$(2)),$$(notdir $$@))

$$(TBIN$(1)_T_$(2)_H_$(3))/rustc$$(X_$(3)): \
Expand Down
6 changes: 3 additions & 3 deletions mk/tools.mk
View file
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ $$(TLIB$(1)_T_$(4)_H_$(3))/$(CFG_LIBRUSTDOC_$(4)): \
| $$(TLIB$(1)_T_$(4)_H_$(3))/
@$$(call E, compile_and_link: $$@)
$$(call REMOVE_ALL_OLD_GLOB_MATCHES_EXCEPT,$$(dir $$@),$(LIBRUSTDOC_GLOB_$(4)),$$(notdir $$@))
$$(STAGE$(1)_T_$(4)_H_$(3)) --out-dir $$(@D) $$< && touch $$@
$$(STAGE$(1)_T_$(4)_H_$(3)) $$(WFLAGS_ST$(1)) --out-dir $$(@D) $$< && touch $$@
$$(call LIST_ALL_OLD_GLOB_MATCHES_EXCEPT,$$(dir $$@),$(LIBRUSTDOC_GLOB_$(4)),$$(notdir $$@))

$$(TBIN$(1)_T_$(4)_H_$(3))/rustdoc$$(X_$(4)): \
Expand All @@ -85,7 +85,7 @@ $$(TLIB$(1)_T_$(4)_H_$(3))/$(CFG_LIBRUSTI_$(4)): \
| $$(TLIB$(1)_T_$(4)_H_$(3))/
@$$(call E, compile_and_link: $$@)
$$(call REMOVE_ALL_OLD_GLOB_MATCHES_EXCEPT,$$(dir $$@),$(LIBRUSTI_GLOB_$(4)),$$(notdir $$@))
$$(STAGE$(1)_T_$(4)_H_$(3)) --out-dir $$(@D) $$< && touch $$@
$$(STAGE$(1)_T_$(4)_H_$(3)) $$(WFLAGS_ST$(1)) --out-dir $$(@D) $$< && touch $$@
$$(call LIST_ALL_OLD_GLOB_MATCHES_EXCEPT,$$(dir $$@),$(LIBRUSTI_GLOB_$(4)),$$(notdir $$@))

$$(TBIN$(1)_T_$(4)_H_$(3))/rusti$$(X_$(4)): \
Expand All @@ -106,7 +106,7 @@ $$(TLIB$(1)_T_$(4)_H_$(3))/$(CFG_LIBRUST_$(4)): \
| $$(TLIB$(1)_T_$(4)_H_$(3))/
@$$(call E, compile_and_link: $$@)
$$(call REMOVE_ALL_OLD_GLOB_MATCHES_EXCEPT,$$(dir $$@),$(LIBRUST_GLOB_$(4)),$$(notdir $$@))
$$(STAGE$(1)_T_$(4)_H_$(3)) --out-dir $$(@D) $$< && touch $$@
$$(STAGE$(1)_T_$(4)_H_$(3)) $$(WFLAGS_ST$(1)) --out-dir $$(@D) $$< && touch $$@
$$(call LIST_ALL_OLD_GLOB_MATCHES_EXCEPT,$$(dir $$@),$(LIBRUST_GLOB_$(4)),$$(notdir $$@))

$$(TBIN$(1)_T_$(4)_H_$(3))/rust$$(X_$(4)): \
Expand Down

5 comments on commit cc56708

@bors
Copy link
Contributor

@bors bors commented on cc56708 Aug 17, 2013

Choose a reason for hiding this comment

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

saw approval from erickt
at erickt@cc56708

@bors
Copy link
Contributor

@bors bors commented on cc56708 Aug 17, 2013

Choose a reason for hiding this comment

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

merging erickt/rust/deny-warnings = cc56708 into auto

@bors
Copy link
Contributor

@bors bors commented on cc56708 Aug 17, 2013

Choose a reason for hiding this comment

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

erickt/rust/deny-warnings = cc56708 merged ok, testing candidate = 7503396

@bors
Copy link
Contributor

@bors bors commented on cc56708 Aug 17, 2013

Choose a reason for hiding this comment

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

all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/955
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/958
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/64
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/64
success: http://buildbot.rust-lang.org/builders/auto-mac-all-opt/builds/955
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/973
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/64
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/64
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/973
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/64
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/64
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt-vg/builds/982
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android/builds/146
success: http://buildbot.rust-lang.org/builders/auto-linux-all-opt/builds/975
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/958
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/64
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/64
success: http://buildbot.rust-lang.org/builders/auto-bsd-64-opt/builds/744

@bors
Copy link
Contributor

@bors bors commented on cc56708 Aug 17, 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 = 7503396

Please sign in to comment.