Skip to content
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

Update rls-data and rls-analysis versions #53440

Closed
staktrace opened this issue Aug 17, 2018 · 6 comments
Closed

Update rls-data and rls-analysis versions #53440

staktrace opened this issue Aug 17, 2018 · 6 comments

Comments

@staktrace
Copy link
Contributor

I'd like to bump rls-analysis and rls-data to the latest available versions (0.16 and 0.18, respectively). Specifically to pick up the changes from rust-dev-tools/rls-analysis#147. It looks like this will need the three-step update process described at /~https://github.com/rust-lang/rust/blob/master/CONTRIBUTING.md#breaking-tools-built-with-the-compiler. I will attempt to create the appropriate patches.

/cc @nrc

@staktrace
Copy link
Contributor Author

Also /cc @emilio since he also made changes to rls-analysis and might be interested in this.

staktrace added a commit to staktrace/rls that referenced this issue Aug 17, 2018
staktrace added a commit to staktrace/rls that referenced this issue Aug 17, 2018
staktrace added a commit to staktrace/rls that referenced this issue Aug 17, 2018
@emilio
Copy link
Contributor

emilio commented Aug 18, 2018

I guess updating it in the compiler before rls was the way to go? I submitted rust-lang/rls#973 but never got an answer about how to do this. Thanks for doing this!

@staktrace
Copy link
Contributor Author

I guess updating it in the compiler before rls was the way to go?

I've never done it before, but that's what the documentation seems to imply. AFAICT step 1 is land it in rustc, step is land the RLS change, and step 3 is update the submodule in rustc to point to the new RLS.

@nrc
Copy link
Member

nrc commented Aug 21, 2018

but never got an answer about how to do this

oh crap, sorry!

step 1 is land it in rustc, step is land the RLS change, and step 3 is update the submodule in rustc to point to the new RLS

That's correct.

GuillaumeGomez pushed a commit to GuillaumeGomez/rust that referenced this issue Aug 22, 2018
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this issue Aug 22, 2018
Update version of rls-data used with save-analysis

This part 1/3 for fixing rust-lang#53440.
staktrace added a commit to staktrace/rust that referenced this issue Aug 23, 2018
@staktrace
Copy link
Contributor Author

Parts 1 and 2 have landed, so I made a PR for part 3. #53649

@staktrace
Copy link
Contributor Author

This is done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants