Conversation
This comment was marked as resolved.
This comment was marked as resolved.
|
I'll try to write an MCP later (unless someone does it for me). This change was motivated after seeing https://internals.rust-lang.org/t/code-compiles-on-playground-but-fails-when-passed-via-stdin-to-rustc/24393 where the different defaults between cargo, playground and rustc caused people to get confused. |
b923a75 to
2a486d3
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
4241536 to
d71c2fa
Compare
|
Some changes occurred in src/tools/compiletest cc @jieyouxu |
|
r? @jieyouxu rustbot has assigned @jieyouxu. Use Why was this reviewer chosen?The reviewer was selected based on:
|
This comment has been minimized.
This comment has been minimized.
d71c2fa to
a1d6de2
Compare
This comment has been minimized.
This comment has been minimized.
|
|
||
| fn main() { | ||
| rustc().input("foo-prev.rs").run(); | ||
| rustc().edition("2015").input("foo-prev.rs").run(); |
There was a problem hiding this comment.
Remark: we could consider default setting edition but allow overriding, but explicit is fine
aa014ef to
047b23e
Compare
--edition, emit a note--edition, emit a message
|
@bors r=jieyouxu |
…ieyouxu When compiling without a specified `--edition`, emit a message Implement rust-lang/compiler-team#1019.
|
💔 I suspect this PR failed tests as part of a rollup After fixing the problem, consider running a try job for the failed job before re-approving. Link to failure: #163456 (comment) |
|
This pull request was unapproved. This PR was contained in a rollup (#163456), which was unapproved. |
…sage Use same wording as cargo does when `package.edition` is unspecified. Do not suggest `--edition=future` on stable.
047b23e to
97cc85e
Compare
|
@JonathanBrouwer There was a change to the behavior of run-make tests since I published this PR (we need to explicitly state that we need std). I'll approve this and mark it |
|
@bors r=jieyouxu rollup=never |
This comment has been minimized.
This comment has been minimized.
When compiling without a specified `--edition`, emit a message Implement rust-lang/compiler-team#1019.
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
💔 Test for 22079c4 failed: CI. Failed job:
|
|
☔ The latest upstream changes (presumably #163465) made this pull request unmergeable. Please resolve the merge conflicts by rebasing. This pull request was unapproved. |
There was a problem hiding this comment.
This is missing target()s but I'd say let's just //@ ignore-cross-compile. This is really a "host tool" behavior.
Implement rust-lang/compiler-team#1019.
View all comments