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

[beta] Remove rls from the workspace and extended builds #43199

Closed
wants to merge 1 commit into from

Conversation

cuviper
Copy link
Member

@cuviper cuviper commented Jul 13, 2017

Its git dependencies don't work when building with vendored crates,
so for now it will just be removed from the workspace and disabled in
the rustbuild rules.

cc #42719

Its git dependencies don't work when building with vendored crates,
so for now it will just be removed from the workspace and disabled in
the rustbuild rules.

cc rust-lang#42719
@rust-highfive
Copy link
Collaborator

r? @alexcrichton

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

@rust-highfive
Copy link
Collaborator

warning Warning warning

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@alexcrichton
Copy link
Member

@bors: r+ p=100

Thanks @cuviper!

@bors
Copy link
Contributor

bors commented Jul 13, 2017

📌 Commit 6d6fccb has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jul 13, 2017

⌛ Testing commit 6d6fccb with merge 4f9ee81fc474ed85defdebd10fbe45607b6d103c...

@bors
Copy link
Contributor

bors commented Jul 13, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member

@bors: r+ p=25

@bors
Copy link
Contributor

bors commented Jul 13, 2017

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.
  • There's another pull request that is currently being tested, blocking this pull request: [beta] backports #43114

@bors
Copy link
Contributor

bors commented Jul 13, 2017

📌 Commit 6d6fccb has been approved by alexcrichton

@alexcrichton alexcrichton added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jul 13, 2017
@alexcrichton
Copy link
Member

I'm going to manually roll this into #43208 to get the new beta out sooner, so closing.

@cuviper cuviper deleted the beta-sans-rls branch October 19, 2017 23:30
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.

4 participants