Skip to content

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Apr 23, 2019

The deny causes a build failure in https://github.com/RalfJung/miri-test-libstd. Since we use -D warnings for rustc builds, warn should be enough to lead to compile errors here, without impeding external builds.

@rust-highfive
Copy link
Contributor

r? @Kimundi

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 23, 2019
@sanxiyn
Copy link
Contributor

sanxiyn commented Apr 24, 2019

This doesn't sound right. Isn't this what --cap-lints is for?

@RalfJung
Copy link
Member Author

This doesn't sound right.

Why not? I have set deny-warnings = false locally, and yet I still get compilation failures from flags like this. So even from a pure rustc perspective, I think "warn" is "more right" than "deny".

@Centril
Copy link
Contributor

Centril commented Apr 28, 2019

r? @Mark-Simulacrum

@RalfJung
Copy link
Member Author

Cc @rust-lang/libs -- needs someone to review.

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented May 13, 2019

📌 Commit 4cb6d1c has been approved by alexcrichton

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 13, 2019
Centril added a commit to Centril/rust that referenced this pull request May 13, 2019
coretest: Downgrade deny to warn

The `deny` causes a build failure in https://github.com/RalfJung/miri-test-libstd. Since we use `-D warnings` for rustc builds, `warn` should be enough to lead to compile errors here, without impeding external builds.
bors added a commit that referenced this pull request May 14, 2019
Rollup of 5 pull requests

Successful merges:

 - #60176 (Explain error when yielding a reference to a local variable)
 - #60201 (coretest: Downgrade deny to warn)
 - #60562 (Add #[doc(hidden)] attribute on compiler generated module.)
 - #60710 (Use `delay_span_bug` for error cases when checking `AnonConst` parent)
 - #60770 (add impl_trait_in_bindings to INCOMPLETE_FEATURES)

Failed merges:

r? @ghost
@bors bors merged commit 4cb6d1c into rust-lang:master May 14, 2019
@RalfJung RalfJung deleted the core-tests branch June 10, 2019 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants