-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Restore lldb build #53951
Restore lldb build #53951
Conversation
commit 6c10142 ("Update LLVM submodule") disabled the lldb build. This patch updates the lldb and clang submodules to once again build against the LLVM that is included in the Rust tree, and reverts the .travis.yml changes from that patch.
(rust_highfive has picked a reviewer for you, use r? to override) |
|
@bors: r+ p=1 Thanks for doing this! Sorry long weekend here so was a bit delayed getting around to it! |
📌 Commit 289da84 has been approved by |
⌛ Testing commit 289da84 with merge f7175d56dd1abae937254e7705ab7564b597c3bd... |
💔 Test failed - status-travis |
The job Click to expand the log.
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 |
The lldb code definitely looks incorrect -- that file has moved -- but I cannot reproduce it in a local build. |
Maybe locally it is picking up an installed lldb header. |
I've fixed the lldb issue and as it is trivial I'm going to see if I can re-r+ this. @bors r=alexcrichton |
@tromey: 🔑 Insufficient privileges: Not in reviewers |
@alexcrichton it will need another r+ from you |
@bors: r+ delegate+ |
✌️ @tromey can now approve this pull request |
📌 Commit 7bb30ff has been approved by |
Restore lldb build commit 6c10142 ("Update LLVM submodule") disabled the lldb build. This patch updates the lldb and clang submodules to once again build against the LLVM that is included in the Rust tree, and reverts the .travis.yml changes from that patch.
☀️ Test successful - status-appveyor, status-travis |
commit 6c10142 ("Update LLVM submodule") disabled the lldb build.
This patch updates the lldb and clang submodules to once again build
against the LLVM that is included in the Rust tree, and reverts the
.travis.yml changes from that patch.