Skip to content

Commit

Permalink
Fix #1480, added -D unused-imports to rustc flags
Browse files Browse the repository at this point in the history
  • Loading branch information
@khodzha
khodzha committed Mar 19, 2014
1 parent caf1ed9 commit 40bf442
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions Makefile.in
View file
Expand Up @@ -30,6 +30,7 @@ B := $(CFG_BUILD_DIR)
MKFILE_DEPS := config.stamp $(call rwildcard,$(S)mk/,*)

CFG_GCCISH_CFLAGS += -DRUST_DEBUG
CFG_RUSTC_FLAGS = -D unused-imports

ifdef CFG_DISABLE_OPTIMIZE
$(info cfg: disabling rustc optimization (CFG_DISABLE_OPTIMIZE))
Expand Down

9 comments on commit 40bf442

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

saw approval from jdm
at khodzha@40bf442

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

merging khodzha/servo/unused_imports = 40bf442 into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

khodzha/servo/unused_imports = 40bf442 merged ok, testing candidate = 238f28e

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

some tests failed:
exception: http://servo-buildbot.pub.build.mozilla.org/builders/mac/builds/1063

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

saw approval from jdm
at khodzha@40bf442

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

merging khodzha/servo/unused_imports = 40bf442 into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

khodzha/servo/unused_imports = 40bf442 merged ok, testing candidate = 371bea7

@bors-servo
Copy link
Contributor

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://servo-buildbot.pub.build.mozilla.org/builders/linux/builds/1111
success: http://servo-buildbot.pub.build.mozilla.org/builders/mac/builds/1066

@bors-servo
Copy link
Contributor

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 = 371bea7

Please sign in to comment.