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 28 pull requests #39199

Merged
merged 63 commits into from
Jan 21, 2017
Merged

Rollup of 28 pull requests #39199

merged 63 commits into from
Jan 21, 2017

Conversation

alexcrichton
Copy link
Member

frewsxcv and others added 30 commits January 1, 2017 11:53
This commit expands the existing x86_64-musl entry in the Travis matrix to also
build/test i586-unknown-linux-gnu and i686-unknown-linux-musl.

cc rust-lang#38531
Closes rust-lang#39053
This is a target that we're shipping today, so this commit adds this matrix
entry to AppVeyor. This reuses the existing i686 MSVC matrix entry as it's
currently finishing about a half hour under two hours, which should hopefully
give it enough extra time to run this test suite.
Somewhere between https://travis-ci.org/rust-lang/rust/jobs/192352185 and
https://travis-ci.org/rust-lang/rust/jobs/192440181 it looks like our
configuration for a newer OSX image was lost as LLDB has reverted itself back to
350. This fix appeared to work for the libc crate so let's see if we can
configure it to work for the rust repo as well.
This expands the `cross` travis matrix entry with a few more targets that our
nightlies are building:

* x86_64-rumprun-netbsd
* arm-unknown-linux-musleabi
* arm-unknown-linux-musleabihf
* armv7-unknown-linux-musleabihf
* mips-unknown-linux-musl
* mipsel-unknown-linux-musl

This commit doesn't compile custom toolchains like our current cross-image does,
but instead compiles musl manually and then compiles libunwind manually (like
x86_64) for use for the ARM targets and just uses openwrt toolchains for the
mips targets.
* Add support for `#[proc_macro]`

* Reactivate `proc_macro` feature and gate `#[proc_macro_attribute]` under it

* Have `#![feature(proc_macro)]` imply `#![feature(use_extern_macros)]`,
error on legacy import of proc macros via `#[macro_use]`
…" pass so proc-macro-attribute feature gate check can use resolve
All our releases are compiled with this, so let's be sure to do so whenever
`DEPLOY` is set. This'll ensure that we don't have dynamic dependencies on
libstdc++ which LLVM depends on, but instead we link it all statically to have
more portable binaries.
Based on a discussion on #rust-beginners the existing note for drain is confusing. This new wording was suggested.
Pass an absolute path, not just the basename.
We'll generate these later in the build process and otherwise they could just
cause spurious failures with files overwriting one another.
@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Contributor

bors commented Jan 20, 2017

📌 Commit 21b112a has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jan 20, 2017

⌛ Testing commit 21b112a with merge ca8479b...

@bors
Copy link
Contributor

bors commented Jan 20, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Contributor

bors commented Jan 20, 2017

📌 Commit 72c3148 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jan 20, 2017

⌛ Testing commit 72c3148 with merge b9b5138...

bors added a commit that referenced this pull request Jan 20, 2017
@bors
Copy link
Contributor

bors commented Jan 21, 2017

💔 Test failed - status-appveyor

@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Contributor

bors commented Jan 21, 2017

📌 Commit 8bee398 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jan 21, 2017

⌛ Testing commit 8bee398 with merge aedb49c...

bors added a commit that referenced this pull request Jan 21, 2017
@bors
Copy link
Contributor

bors commented Jan 21, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing aedb49c 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.