-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Rollup of 12 pull requests #47350
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Rollup of 12 pull requests #47350
Conversation
kennytm
commented
Jan 11, 2018
- Successful merges: Update musl to 1.1.18 #47283, Update jobserver to 0.1.9 #47288, Skip linker-output-non-utf8 test on Apple #47289, Treat #[path] files as mod.rs files #47298, Use copy/clone closures to simplify calendar test #47305, Fix panic strings. #47307, fix typo rwlock.rs #47310, resolve type and region variables in "NLL dropck" #47322, Pre-allocate in fs::read and fs::read_string #47324, Use the new fs_read_write functions in rustc internals #47328, Fix typo. #47340, Add tests to fixed issues. #47344
- Failed merges:
According to http://www.musl-libc.org/download.html: This release corrects regressions in glob() and armv4t build failure introduced in the previous release, and includes an important bug fix for posix_spawnp in the presence of a large PATH environment variable.
Fix for `ENOSYS` when calling `pipe2`, rust-lang/jobserver-rs#5. r? @alexcrichton
This test fails on APFS filesystems with the following error: mkdir: /Users/ryan/Code/rust/build/x86_64-apple-darwin/test/run-make/linker-output-non-utf8.stage2-x86_64-apple-darwin/zzz�: Illegal byte sequence This is due to APFS now requiring that all paths are valid UTF-8. As APFS will be the default filesystem for all new Darwin-based systems the most straightforward fix is to skip this test on Darwin as well as Windows.
The Windows situation is more complicated than I realised
Hi. Fixed typo: contained -> content
- Fix panic string in `check_ast_crate`.
Closes rust-lang#36792. Closes rust-lang#38091. Closes rust-lang#39687. Closes rust-lang#42148. Closes rust-lang#42956.
Update musl to 1.1.18 According to http://www.musl-libc.org/download.html: This release corrects regressions in glob() and armv4t build failure introduced in the previous release, and includes an important bug fix for posix_spawnp in the presence of a large PATH environment variable.
…hton Update jobserver to 0.1.9 Fix for `ENOSYS` when calling `pipe2`, rust-lang/jobserver-rs#5. r? @alexcrichton
…-test-on-apple, r=kennytm Skip linker-output-non-utf8 test on Apple This test fails on APFS filesystems with the following error: ```shell mkdir: /Users/ryan/Code/rust/build/x86_64-apple-darwin/test/run-make/linker-output-non-utf8.stage2-x86_64-apple-darwin/zzz�: Illegal byte sequence ``` The mkdir does succeed on an HFS+ volume mounted on the same system: ```shell $ mkdir zzz$$'\xff' $ ls zzz47432\xff ``` This is due to APFS now requiring that all paths are valid UTF-8. As APFS will be the default filesystem for all new Darwin-based systems the most straightforward fix is to skip this test on Darwin as well as Windows.
Treat #[path] files as mod.rs files Fixes rust-lang#46936, cc @briansmith, @SergioBenitez, @nikomatsakis. This (insta-stable) change treats files included via `#[path = "bla.rs"] mod foo;` as though they were `mod.rs` files. Namely, it allows them to include `mod` statements and looks for the child modules in sibling directories, rather than in relative `modname/childmodule.rs` files as happens for non-`mod.rs` files. This change makes the `non_modrs_mods` feature backwards compatible with the existing usage in https://github.com/briansmith/ring, several versions of which are currently broken in beta. If we decide to merge, this change should be backported to beta. cc rust-lang#37872 r? @jseyfried
…ddyb Use copy/clone closures to simplify calendar test Split out from rust-lang#47304 r? @eddyb
…nnytm Fix panic strings. - Fix panic string in `check_ast_crate`. - Update panic string for Duration subtraction on overflow/underflow. Not sure if the changes to `Duration` are helpful/needed. Mostly just a nit. Otherwise this is just a one character change :) On another note: I hit the panic in `check_ast_crate` when compiling [m-labs/smoltcp] with the following: ``` cargo test --doc --no-default-features --features "std socket-raw"` ``` [m-labs/smoltcp]: https://github.com/m-labs/smoltcp
fix typo rwlock.rs Hi. Fixed typo: contained -> content
resolve type and region variables in "NLL dropck" Fixes rust-lang#47022. r? @pnkfelix
Pre-allocate in fs::read and fs::read_string This is a simpler alternative to rust-lang#46340 and rust-lang#45928, as requested by the libs team.
Use the new fs_read_write functions in rustc internals Uses `fs::read` and `fs::write` (added by rust-lang#45837) where appropriate, to simplify code and dog-food these new APIs. This also improves performance, when combined with rust-lang#47324.
Add tests to fixed issues. Closes rust-lang#36792. Closes rust-lang#38091. Closes rust-lang#39687. Closes rust-lang#42148. Closes rust-lang#42956.
r? @estebank (rust_highfive has picked a reviewer for you, use r? to override) |
@bors r+ p=12 |
📌 Commit 5bd7d69 has been approved by |
⌛ Testing commit 5bd7d69 with merge 12b5c2dbb77cd6bb2e2b1ec6dff5a59d3832b7e3... |
💔 Test failed - status-appveyor |
⌛ Testing commit 5bd7d69 with merge 9bceb61b76096fe47725eff18168234980709c12... |
💔 Test failed - status-travis |
@bors retry GitHub was down. |
💔 Test failed - status-appveyor |
3 hour timeout but on check-aux 🤔 The stage1-rustc compilation time is 15 minutes longer than the previous retry with no reason. @bors retry |
⌛ Testing commit 5bd7d69 with merge dc9a79af33018ad9a135e8032e4ee37fa9e26e54... |
💔 Test failed - status-travis |
@bors retry 30-minutes no output from
|
💔 Test failed - status-appveyor |
@bors retry 3 hour timeout on i686-pc-windows-gnu |
⌛ Testing commit 5bd7d69 with merge 620db9f4cd39921164a4d1005fd23ae08a12deda... |
💔 Test failed - status-appveyor |