-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Rollup of 4 pull requests #34358
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
Rollup of 4 pull requests #34358
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
@bors r+ p=10 force |
📌 Commit 5bca023 has been approved by |
⌛ Testing commit 5bca023 with merge 3ba605f... |
💔 Test failed - auto-mac-64-opt-rustbuild |
@bors r+ p=10 force |
📌 Commit e3740b3 has been approved by |
@bors: retry force clean
|
⌛ Testing commit e3740b3 with merge fda55b1... |
We no C++ and an incredibly small amount of C code as part of the build, so there's not really much need for us to strictly check the version of compilers as we're not really stressing anything. LLVM is a pretty huge chunk of C++ but it should be the responsibility of LLVM to ensure that it can build with a particular clang/gcc version, not ours (as this logic changes over time). These version checks seem to basically just by us a regular stream of PRs every six weeks or so when a new version is releases, so they're not really buying us much. As a result, remove them and we can add then back piecemeal perhaps as a blacklist if we really need to.
@bors: retry force clean
|
⌛ Testing commit e3740b3 with merge 50d0470... |
- src links/redirects to extern fn from another crate had an extra '/'. - src links to `pub use` of a crate module had an extra '/'. - src links to renamed reexports from another crate used the new name for the link but should use the original name.
💔 Test failed - auto-win-gnu-32-nopt-t |
This is mostly just a documentation fix as I don't think stability attributes have any effect on macros.
Implement Binary, Octal, LowerHex and UpperHex for Wrapping<T> Fixes: rust-lang#33659
Fix overflow error in thread::sleep Fixes rust-lang#34330 I added a test to have a more clear error inside the function. Since `time_t` is `i64` and we expect `u64`, maybe we should changed the awaited type?
Add regression test for rust-lang#23281 Closes rust-lang#23281
…r=brson configure: Remove clang version checks We no C++ and an incredibly small amount of C code as part of the build, so there's not really much need for us to strictly check the version of compilers as we're not really stressing anything. LLVM is a pretty huge chunk of C++ but it should be the responsibility of LLVM to ensure that it can build with a particular clang/gcc version, not ours (as this logic changes over time). These version checks seem to basically just by us a regular stream of PRs every six weeks or so when a new version is releases, so they're not really buying us much. As a result, remove them and we can add then back piecemeal perhaps as a blacklist if we really need to.
…ichton rustdoc: Fix a couple of issues with src links to external crates - src links/redirects to extern fn from another crate had an extra '/'. - src links to `pub use` of a crate module had an extra '/'. - src links to renamed reexports from another crate used the new name for the link but should use the original name. Fixes: rust-lang#34274
…r=eddyb don't warn on casting byte strs to slices r? @durka
Mark concat_idents! unstable This is mostly just a documentation fix as I don't think stability attributes have any effect on macros. [before](https://doc.rust-lang.org/nightly/std/macro.concat_idents!.html) [after](https://ollie27.github.io/rust_doc_test/std/macro.concat_idents!.html)
std::thread::panicking
. #34313, Add short summaries to btree modules #34335, fix build for clang 8.0 #34340, DocumentCStr::as_ptr
dangers. #34356