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.

Sign up for GitHub

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

Jump to bottom

The rls-preview component on beta is "corrupted" #44270

Closed
alexcrichton opened this issue Sep 2, 2017 · 1 comment
Closed

The rls-preview component on beta is "corrupted" #44270

alexcrichton opened this issue Sep 2, 2017 · 1 comment
Assignees
@nrc
Labels
C-bug Category: This is a bug. regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-dev-tools Relevant to the dev-tools subteam, which will review and decide on the PR/issue.
Milestone
1.21

Comments

@alexcrichton
Copy link
Member

With the second release of the beta toolchain we've got the rls-preview component now (not the rls) and it looks like it's almost working! The only piece that seems off is:

$ rustup component add rls-preview --toolchain beta
info: downloading component 'rls-preview'
info: installing component 'rls-preview'
info: rolling back changes
error: component manifest for 'rls-preview' is corrupt

cc @nrc

Note that I've tagged this as a beta regressoin but it's not actually a regression, it's just there to make sure we keep looking at it.
@alexcrichton alexcrichton added the regression-from-stable-to-beta Performance or correctness regression from stable to beta. label Sep 2, 2017
@alexcrichton alexcrichton added this to the 1.21 milestone Sep 2, 2017
@nrc nrc self-assigned this Sep 3, 2017
nrc added a commit to nrc/rust that referenced this issue Sep 5, 2017
@nrc
Attempt to fix the component manifest problem for rls-preview
1ffc3dd 
cc rust-lang#44270
@nrc nrc mentioned this issue Sep 5, 2017
Closed
alexcrichton pushed a commit to nrc/rust that referenced this issue Sep 5, 2017
@nrc @alexcrichton
Attempt to fix the component manifest problem for rls-preview
0f2a65a 
cc rust-lang#44270
nrc added a commit to nrc/rust that referenced this issue Sep 5, 2017
@nrc
Attempt to fix the component manifest problem for rls-preview
c67207d 
cc rust-lang#44270
@nrc nrc mentioned this issue Sep 5, 2017
Merged
@nrc nrc mentioned this issue Sep 5, 2017
Merged
@alexcrichton alexcrichton added A-rls C-bug Category: This is a bug. T-dev-tools Relevant to the dev-tools subteam, which will review and decide on the PR/issue. labels Sep 7, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this issue Sep 11, 2017
@frewsxcv
Rollup merge of rust-lang#44356 - nrc:rls-component-manifest, r=@alex…
86402ad 
…crichton

Attempt to fix the component manifest problem for rls-preview

cc rust-lang#44270

See rust-lang#44270
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this issue Sep 13, 2017
@GuillaumeGomez
Rollup merge of rust-lang#44356 - nrc:rls-component-manifest, r=@alex…
5ebf810 
…crichton

Attempt to fix the component manifest problem for rls-preview

cc rust-lang#44270

See rust-lang#44270
frewsxcv added a commit to frewsxcv/rust that referenced this issue Sep 14, 2017
@frewsxcv
Rollup merge of rust-lang#44356 - nrc:rls-component-manifest, r=@alex…
0952276 
…crichton

Attempt to fix the component manifest problem for rls-preview

cc rust-lang#44270

See rust-lang#44270
nrc added a commit to nrc/rust that referenced this issue Sep 14, 2017
@nrc
Attempt to fix the component manifest problem for rls-preview
e25513c 
cc rust-lang#44270
nrc added a commit to nrc/rust that referenced this issue Sep 14, 2017
@nrc
Attempt to fix the component manifest problem for rls-preview
39c9a3d 
cc rust-lang#44270
bors added a commit that referenced this issue Sep 14, 2017
@bors
Auto merge of #44357 - nrc:rls-manifest-beta-2, r=alexcrichton
9637179 
[beta] Attempt to fix the component manifest problem for rls-preview

cc #44270

see #44352
alexcrichton pushed a commit to nrc/rust that referenced this issue Sep 14, 2017
@nrc @alexcrichton
Attempt to fix the component manifest problem for rls-preview
45b426c 
cc rust-lang#44270
bors added a commit that referenced this issue Sep 15, 2017
@bors
Auto merge of #44357 - nrc:rls-manifest-beta-2, r=alexcrichton
ea1fd7d 
[beta] Attempt to fix the component manifest problem for rls-preview

cc #44270

see #44352
@alexcrichton
Copy link
Member Author

Fixed in #44357!

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this issue Sep 15, 2017
@GuillaumeGomez
Rollup merge of rust-lang#44356 - nrc:rls-component-manifest, r=@alex…
0c39a5a 
…crichton

Attempt to fix the component manifest problem for rls-preview

cc rust-lang#44270

See rust-lang#44270
alexcrichton added a commit to alexcrichton/rust that referenced this issue Sep 16, 2017
@alexcrichton
Rollup merge of rust-lang#44356 - nrc:rls-component-manifest, r=@alex…
fb954ba 
…crichton

Attempt to fix the component manifest problem for rls-preview

cc rust-lang#44270

See rust-lang#44270
frewsxcv added a commit to frewsxcv/rust that referenced this issue Sep 16, 2017
@frewsxcv
Rollup merge of rust-lang#44356 - nrc:rls-component-manifest, r=@alex…
ac3ac7c 
…crichton

Attempt to fix the component manifest problem for rls-preview

cc rust-lang#44270

See rust-lang#44270
alexcrichton added a commit to alexcrichton/rust that referenced this issue Sep 17, 2017
@alexcrichton
Rollup merge of rust-lang#44356 - nrc:rls-component-manifest, r=@alex…
48193c2 
…crichton

Attempt to fix the component manifest problem for rls-preview

cc rust-lang#44270

See rust-lang#44270
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nrc nrc

Labels
C-bug Category: This is a bug. regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-dev-tools Relevant to the dev-tools subteam, which will review and decide on the PR/issue.
Projects
None yet
Milestone
1.21
Development

No branches or pull requests
2 participants
@alexcrichton @nrc