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 15 pull requests #32960

Merged
merged 32 commits into from
Apr 15, 2016
Merged

Rollup of 15 pull requests #32960

merged 32 commits into from
Apr 15, 2016

Conversation

Ms2ger and others added 30 commits March 31, 2016 14:54
The path `/etc/rustc/` is not the default last entry in
RUST_TARGET_PATH. This was in RFC131 but was never implemented in rustc
so it was removed as part of rust-lang#31117 and rust-lang/rfcs#1473.

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
The original is correct, but a bit misleading.
This test tests a message that is only displayed on nightly.
The next project refers to the dining-philosophers problem
https://github.com/rust-lang/rust/blob/27a1834ce522e3ec7fe4726b1661de16ee30c503/src/doc/book/dining-philosophers.md

which was removed in
rust-lang@0c6c34d

so removing reference to that project from book/guessing-game
Stop re-exporting check_match::Constructor.
…chton

Don't read past limit for in BufRead instance of Take

Similar to `Read::read`, `BufRead::fill_buf` impl of `Take` should not call `inner.fill_buf` if the limit is already reached.
…hton

librustc_back: fix incorrect comment about RUST_TARGET_PATH

The path `/etc/rustc/` is not the default last entry in
RUST_TARGET_PATH. This was in RFC131 but was never implemented in rustc
so it was removed as part of rust-lang#31117 and rust-lang/rfcs#1473.

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
Add rustbuild option to use Ninja for LLVM build

This change adds support for a `ninja` option in the `[llvm]` section of rustbuild's `config.toml`. When `true`, the option enables use of the Ninja build tool. Note that this change does not add support for Ninja to the old makefile based build system.

Closes rust-lang#32809

r? @alexcrichton
Remove an extra command from the usage message
Add test for is_char_boundary

Add test for is_char_boundary

Apparently there was no test for this method. This test is rather simple, not exhaustive.
Fix conflicting link identifiers

Caused "Errors for non-exhaustive match patterns now list up to 3 missing variants while also indicating the total number of missing variants if more than 3." to link to "libsyntax: Restrict where non-inline modules can appear (fixes rust-lang#29765)"
…bnik

Clarify try! doc example

The original is correct, but a bit misleading.

r? @steveklabnik
Fix beta branch

This test tests a message that is only displayed on nightly.

Still testing this locally, but I suspect it's correct. r? @alexcrichton
…roject-in-guessing-game-doc, r=Manishearth

Doc fix: Do not mention next project in book/guessing-game

fixes rust-lang#32936
@rust-highfive
Copy link
Collaborator

r? @arielb1

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

@steveklabnik
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Contributor

bors commented Apr 14, 2016

📌 Commit 7e2302b has been approved by steveklabnik

@steveklabnik steveklabnik assigned steveklabnik and unassigned arielb1 Apr 14, 2016
@bors
Copy link
Contributor

bors commented Apr 14, 2016

⌛ Testing commit 7e2302b with merge a9d294c...

@bors
Copy link
Contributor

bors commented Apr 14, 2016

💔 Test failed - auto-win-gnu-32-opt

@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Contributor

bors commented Apr 14, 2016

⌛ Testing commit 7e2302b with merge 2174bd9...

bors added a commit that referenced this pull request Apr 14, 2016
@bors bors merged commit 7e2302b into rust-lang:master Apr 15, 2016
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.