New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stabilize edition 2018; also updates Clippy, RLS and Cargo #54057

Merged
merged 4 commits into from Sep 9, 2018

Conversation

Projects
None yet
6 participants
@matthiaskrgr
Copy link
Contributor

matthiaskrgr commented Sep 8, 2018

Supersedes #53999 , #53935
Clippy build was failing there because crate_visibility_modifier feature was taken out of edition 2018 and clippy used it.
The clippy update enables the corresponding feature explicitly.

r? @Mark-Simulacrum

@rust-highfive

This comment has been minimized.

Copy link
Collaborator

rust-highfive commented Sep 8, 2018

⚠️ Warning ⚠️

  • These commits modify submodules.
@matthiaskrgr

This comment has been minimized.

Copy link
Contributor Author

matthiaskrgr commented Sep 8, 2018

Urgh, looks like rls needs to be fixed as well....

@matthiaskrgr matthiaskrgr reopened this Sep 8, 2018

@matthiaskrgr matthiaskrgr changed the title Stabilize edition 2018 and update clippy submodule [waits on rls upstream] Stabilize edition 2018 and update clippy,rls (<- todo) submodules Sep 8, 2018

@Mark-Simulacrum Mark-Simulacrum force-pushed the matthiaskrgr:stabilize-edition-plus-clippy branch from c0dfe1d to 3e56dec Sep 8, 2018

@Mark-Simulacrum Mark-Simulacrum changed the title [waits on rls upstream] Stabilize edition 2018 and update clippy,rls (<- todo) submodules Stabilize edition 2018; also updates Clippy and RLS Sep 8, 2018

@Mark-Simulacrum

This comment has been minimized.

Copy link
Member

Mark-Simulacrum commented Sep 8, 2018

@bors r=alexcrichton (carrying over approval from #53999)

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Sep 8, 2018

📌 Commit 3e56dec has been approved by alexcrichton

@Mark-Simulacrum

This comment has been minimized.

Copy link
Member

Mark-Simulacrum commented Sep 8, 2018

@bors p=5

Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Sep 8, 2018

Rollup merge of rust-lang#54057 - matthiaskrgr:stabilize-edition-plus…
…-clippy, r=alexcrichton

Stabilize edition 2018; also updates Clippy and RLS

Supersedes rust-lang#53999
Clippy build was failing there because crate_visibility_modifier feature was taken out of edition 2018 and clippy used it.
The clippy update enables the corresponding feature explicitly.

r? @Mark-Simulacrum

bors added a commit that referenced this pull request Sep 8, 2018

Auto merge of #54068 - Mark-Simulacrum:rollup, r=Mark-Simulacrum
Rollup of 11 pull requests

Successful merges:

 - #53851 (Limit the promotion of const fns to the libstd and the `rustc_promotable` attribute)
 - #53949 (Improve messages for un-closed delimiter errors)
 - #53960 (Fix incorrect outer function type parameter message)
 - #53988 (rustc_resolve: only prepend CrateRoot to a non-keyword segment.)
 - #53995 (Too many errors for incorrect move in loop with NLL enabled)
 - #53998 (rustc_codegen_llvm: don't assume offsets are always aligned.)
 - #54000 (Allow named lifetimes in async functions.)
 - #54011 (rustc_resolve: inject `uniform_paths` canary always on Rust 2018.)
 - #54024 (Fix compiling some rustc crates to wasm)
 - #54057 (Stabilize edition 2018; also updates Clippy and RLS)
 - #54064 (`&CStr`, not `CStr`, is the counterpart of `&str`)

Failed merges:

 - #54031 (A few cleanups and minor improvements to rustc_passes)

r? @ghost
@bors

This comment has been minimized.

Copy link
Contributor

bors commented Sep 8, 2018

⌛️ Testing commit 3e56dec with merge 3e3d862...

bors added a commit that referenced this pull request Sep 8, 2018

Auto merge of #54057 - matthiaskrgr:stabilize-edition-plus-clippy, r=…
…alexcrichton

Stabilize edition 2018; also updates Clippy and RLS

Supersedes #53999
Clippy build was failing there because crate_visibility_modifier feature was taken out of edition 2018 and clippy used it.
The clippy update enables the corresponding feature explicitly.

r? @Mark-Simulacrum
@bors

This comment has been minimized.

Copy link
Contributor

bors commented Sep 9, 2018

💔 Test failed - status-appveyor

@matthiaskrgr

This comment has been minimized.

Copy link
Contributor Author

matthiaskrgr commented Sep 9, 2018

Looks like rls fails to build..?

   --> tools\rls\src\build\plan.rs:189:80
    |
189 |                 *target.kind() == TargetKind::CustomBuild && target.src_path().is_path()
    |                                                                                ^^^^^^^
error[E0599]: no method named `path` found for type `&std::path::Path` in the current scope
   --> tools\rls\src\build\plan.rs:191:56
    |
191 |             .map(|(key, unit)| (unit.target.src_path().path(), key.clone()))
    |                                                        ^^^^
error[E0599]: no method named `path` found for type `&std::path::Path` in the current scope
   --> tools\rls\src\build\plan.rs:202:26
    |
202 |                         .path()
    |                          ^^^^
error[E0599]: no method named `path` found for type `&std::path::Path` in the current scope
  --> tools\rls\src\project_model.rs:82:44
   |
82 |                     .map(|t| (t.src_path().path().to_owned(), t.name().replace('-', "_"))),
   |                                            ^^^^
error: aborting due to 4 previous errors
For more information about this error, try `rustc --explain E0599`.
[RUSTC-TIMING] rls test:false 4.825
error: Could not compile `rls`.
@ehuss

This comment has been minimized.

Copy link
Contributor

ehuss commented Sep 9, 2018

@matthiaskrgr I think #53935 needs to be merged first.

@matthiaskrgr

This comment has been minimized.

Copy link
Contributor Author

matthiaskrgr commented Sep 9, 2018

@ehuss Hm, why?

@ehuss

This comment has been minimized.

Copy link
Contributor

ehuss commented Sep 9, 2018

RLS requires the update to Cargo (due to rust-lang/rls#1048), and Cargo requires the update to RLS (since RLS is not allowed to be broken).

@matthiaskrgr

This comment has been minimized.

Copy link
Contributor Author

matthiaskrgr commented Sep 9, 2018

Uff....
I might have to try pulling in a cargo update into this PR then..

@matthiaskrgr matthiaskrgr force-pushed the matthiaskrgr:stabilize-edition-plus-clippy branch from 3e56dec to 801e613 Sep 9, 2018

@matthiaskrgr

This comment has been minimized.

Copy link
Contributor Author

matthiaskrgr commented Sep 9, 2018

@ehuss I tried merging the cargo changes of 53953 now.

@matthiaskrgr matthiaskrgr changed the title Stabilize edition 2018; also updates Clippy and RLS Stabilize edition 2018; also updates Clippy, RLS and Cargo Sep 9, 2018

@rust-highfive

This comment has been minimized.

Copy link
Collaborator

rust-highfive commented Sep 9, 2018

The job x86_64-gnu-llvm-5.0 of your PR failed on Travis (raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.

[00:05:32] travis_fold:start:tidy
travis_time:start:tidy
tidy check
[00:05:33] tidy error: /checkout/src/bootstrap/test.rs:242: trailing whitespace
[00:05:34] some tidy checks failed
[00:05:34] 
[00:05:34] 
[00:05:34] command did not execute successfully: "/checkout/obj/build/x86_64-unknown-linux-gnu/stage0-tools-bin/tidy" "/checkout/src" "/checkout/obj/build/x86_64-unknown-linux-gnu/stage0/bin/cargo" "--no-vendor" "--quiet"
[00:05:34] 
[00:05:34] 
[00:05:34] failed to run: /checkout/obj/build/bootstrap/debug/bootstrap test src/tools/tidy
[00:05:34] Build completed unsuccessfully in 0:00:54
[00:05:34] Build completed unsuccessfully in 0:00:54
[00:05:34] Makefile:79: recipe for target 'tidy' failed
[00:05:34] make: *** [tidy] Error 1

The command "stamp sh -x -c "$RUN_SCRIPT"" exited with 2.
travis_time:start:128eefce
$ date && (curl -fs --head https://google.com | grep ^Date: | sed 's/Date: //g' || true)
---
travis_time:end:0aeb74b0:start=1536488984423324351,finish=1536488984429906725,duration=6582374
travis_fold:end:after_failure.3
travis_fold:start:after_failure.4
travis_time:start:018c6f44
$ ln -s . checkout && for CORE in obj/cores/core.*; do EXE=$(echo $CORE | sed 's|obj/cores/core\.[0-9]*\.!checkout!\(.*\)|\1|;y|!|/|'); if [ -f "$EXE" ]; then printf travis_fold":start:crashlog\n\033[31;1m%s\033[0m\n" "$CORE"; gdb -q -c "$CORE" "$EXE" -iex 'set auto-load off' -iex 'dir src/' -iex 'set sysroot .' -ex bt -ex q; echo travis_fold":"end:crashlog; fi; done || true
travis_fold:end:after_failure.4
travis_fold:start:after_failure.5
travis_time:start:04ad16ce
travis_time:start:04ad16ce
$ cat ./obj/build/x86_64-unknown-linux-gnu/native/asan/build/lib/asan/clang_rt.asan-dynamic-i386.vers || true
cat: ./obj/build/x86_64-unknown-linux-gnu/native/asan/build/lib/asan/clang_rt.asan-dynamic-i386.vers: No such file or directory
travis_fold:end:after_failure.5
travis_fold:start:after_failure.6
travis_time:start:1eb2d2e3
$ dmesg | grep -i kill

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @TimNN. (Feature Requests)

merge cargo changes done by #53935
(done by matthiaskrgr, but I authored ehuss)

@matthiaskrgr matthiaskrgr force-pushed the matthiaskrgr:stabilize-edition-plus-clippy branch from 801e613 to 60be068 Sep 9, 2018

@ehuss

This comment has been minimized.

Copy link
Contributor

ehuss commented Sep 9, 2018

@ehuss I tried merging the cargo changes of 53953 now.

👍 hopefully this can get through the queue before tuesday 😄

@Mark-Simulacrum

This comment has been minimized.

Copy link
Member

Mark-Simulacrum commented Sep 9, 2018

@bors r+ p=5

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Sep 9, 2018

📌 Commit 60be068 has been approved by Mark-Simulacrum

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Sep 9, 2018

⌛️ Testing commit 60be068 with merge f50b775...

bors added a commit that referenced this pull request Sep 9, 2018

Auto merge of #54057 - matthiaskrgr:stabilize-edition-plus-clippy, r=…
…Mark-Simulacrum

Stabilize edition 2018; also updates Clippy, RLS and Cargo

Supersedes #53999 , #53935
Clippy build was failing there because crate_visibility_modifier feature was taken out of edition 2018 and clippy used it.
The clippy update enables the corresponding feature explicitly.

r? @Mark-Simulacrum
@bors

This comment has been minimized.

Copy link
Contributor

bors commented Sep 9, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: Mark-Simulacrum
Pushing f50b775 to master...

@bors bors merged commit 60be068 into rust-lang:master Sep 9, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details

@bors bors referenced this pull request Sep 9, 2018

Closed

Update Cargo and RLS #53935

@kennytm kennytm added the relnotes label Sep 13, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment