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

Rollup of 24 pull requests #47392

Merged
merged 55 commits into from
Jan 12, 2018
Merged

Rollup of 24 pull requests #47392

merged 55 commits into from
Jan 12, 2018

Conversation

Diggsey and others added 30 commits December 28, 2017 13:27
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`.
When gluing two tokens, the second of which is joint, the result should also be
joint.
This fixes an issue with joining three `Dot` tokens to make a `DotDotDot` - the
intermediate `DotDot` would not be joint and therefore we would not attempt to
glue the last `Dot` token, yielding `.. .` instead of `...`.
Just like on UNIX systems, we need to use sysconf() to obtain the number
of CPUs. Extend the existing cfg()'s to match CloudABI as well.
@kennytm
Copy link
Member Author

kennytm commented Jan 12, 2018

@bors r+ p=25

Let's see how many retries are needed this time 😓

@bors
Copy link
Contributor

bors commented Jan 12, 2018

📌 Commit e024f40 has been approved by kennytm

@bors
Copy link
Contributor

bors commented Jan 12, 2018

⌛ Testing commit e024f40d8ba2f4e5663e1dc820aff1e459b77207 with merge 600b33cce4b7408d818c8dc0885300502e7154be...

@bors
Copy link
Contributor

bors commented Jan 12, 2018

💔 Test failed - status-appveyor

@kennytm kennytm changed the title Rollup of 25 pull requests Rollup of 24 pull requests Jan 12, 2018
Glued tokens can themselves be joint.

When gluing two tokens, the second of which is joint, the result should also be
joint.
This fixes an issue with joining three `Dot` tokens to make a `DotDotDot` - the
intermediate `DotDot` would not be joint and therefore we would not attempt to
glue the last `Dot` token, yielding `.. .` instead of `...`.

r? @jseyfried
Make libtest build on CloudABI.

Just like on UNIX systems, we need to use sysconf() to obtain the number
of CPUs. Extend the existing cfg()'s to match CloudABI as well.
…omez

Fix docs for OsStr

At present, there are two small issues with the [docs](https://doc.rust-lang.org/std/ffi/struct.OsStr.html) for std::ffi::OsStr:
- The docs say "OsStr is to OsString as String is to &str: the former in each pair are borrowed references; the latter are owned strings.". The latter pair is mixed up: String is the owned variant whereas &str is the borrowed reference.
- The doc links to String and &str are broken and render as [String] and [&str].

This PR fixes these issues.
Re-enable num tests on wasm

Issue rust-lang#42630 was closed but the tests are still ignored, supposedly they should pass now.
Fix examples of Duration::subsec_millis and Duration::subsec_micros

Update examples of `Duration::subsec_millis` and `Duration::subsec_micros`, because they are not for these two methods actually.
Ignore CTFE errors while lowering patterns

Closes rust-lang#43105.

r? @eddyb
@kennytm
Copy link
Member Author

kennytm commented Jan 12, 2018

@bors r +

Removed #47334.

@bors
Copy link
Contributor

bors commented Jan 12, 2018

📌 Commit c12eabf has been approved by kennytm

@bors
Copy link
Contributor

bors commented Jan 12, 2018

⌛ Testing commit c12eabf with merge 51b0b37...

bors added a commit that referenced this pull request Jan 12, 2018
@bors
Copy link
Contributor

bors commented Jan 12, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: kennytm
Pushing 51b0b37 to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup 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.