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 9 pull requests #45204

Closed
wants to merge 24 commits into from
Closed

Rollup of 9 pull requests #45204

wants to merge 24 commits into from

Conversation

leoyvens and others added 23 commits October 2, 2017 22:04
It's the only use of the function.
Brings in a bugfix to be compatible with the master branch of `make` where
jobserver fds are set in nonblocking mode
…ng#44433.

The main goal here is to use FreeBSD's normal libc++, instead of
statically linking the libstdc++ packaged with GCC, because that
libstdc++ has bugs that cause rustc to deadlock inside LLVM.

But the easiest way to use libc++ is to switch the build from GCC to
Clang, and the Clang package in the Ubuntu image already knows how to
cross-compile (given a sysroot and preferably cross-binutils), so the
toolchain script now uses that instead of building a custom compiler.

This also de-duplicates the `build-toolchain.sh` script.
Paths to object files generated from them were too long and caused errors
output message is shown in another 'help:' block

line with +100 columns formatted

test adjusted
…, r=jseyfried

Inline eq_slice into str::eq

It's the only use of the function.
…ackler

Update the `jobserver` crate

Brings in a bugfix to be compatible with the master branch of `make` where
jobserver fds are set in nonblocking mode
…chton

Re-do the FreeBSD cross-builds to use Clang and libc++.  Fixes rust-lang#44433.

The main goal here is to use FreeBSD's normal libc++, instead of
statically linking the libstdc++ packaged with GCC, because that
libstdc++ has bugs that cause rustc to deadlock inside LLVM.

But the easiest way to use libc++ is to switch the build from GCC to
Clang, and the Clang package in the Ubuntu image already knows how to
cross-compile (given a sysroot and preferably cross-binutils), so the
toolchain script now uses that instead of building a custom compiler.

This also de-duplicates the `build-toolchain.sh` script.
Fix rustc documentation typo

I'm new to rustc but this looks like a typo.
…ichton

Fix PEP8 style issues in bootstrap code

This fixes PEP8 style issues (other than line-length) in the bootstrap Python code.

The most important fix is in the `set` function where the code was indented with 6 spaces instead of 4.
Better compile error output when using arguments instead of types

Following @estebank sugestion on issue rust-lang#18945 (comment)
…labnik

Documented a few more unstable feature gates.

unboxed closures, fn_traits, and OIBIT. hope these are decent!
Shorten some test names

An immediate fix for the first issue in rust-lang#45103
r? @alexcrichton
@rust-highfive
Copy link
Collaborator

r? @aturon

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

@kennytm
Copy link
Member Author

kennytm commented Oct 11, 2017

@bors r+ p=9

@bors
Copy link
Contributor

bors commented Oct 11, 2017

📌 Commit 31bed4a has been approved by kennytm

Prevent fmt::Arguments from being shared across threads

Fixes rust-lang#45197

This is a **breaking change**! Without doing this it's very easy to create race conditions.

There's probably a way to do this without breaking valid use cases, but it would require quite an overhaul of the formatting machinery.
@bors
Copy link
Contributor

bors commented Oct 11, 2017

📌 Commit b03a244 has been approved by kennytm

@Mark-Simulacrum
Copy link
Member

Deapproving because of #45198 -- we want team sign off on it.

@bors r-

@kennytm kennytm deleted the rollup branch October 11, 2017 17:09
@Centril Centril added the rollup A PR which is a rollup label Oct 24, 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.