Skip to content

Rollup of 7 pull requests #29883

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

Merged
merged 14 commits into from
Nov 17, 2015
Merged

Rollup of 7 pull requests #29883

merged 14 commits into from
Nov 17, 2015

Conversation

petrochenkov and others added 14 commits November 16, 2015 21:16
On distros that use i486 or i586 in their CHOST, Rust will fail to build
because it is not handling i486 or i586 like i686 is handled. This
changes the match to do work for all instances of i?86 instead of just
i686. The Yocto Project still uses i586 as a target.

Signed-off-by: Doug Goldstein <[email protected]>
Change conditional perfect to past perfect.
On distros that use i486 or i586 in their CHOST, Rust will fail to build
because it is not handling i486 or i586 like i686 is handled. This
changes the match to do work for all instances of i?86 instead of just
i686. The Yocto Project still uses i586 as a target.

Signed-off-by: Doug Goldstein <[email protected]>
…chton

With this change the build on FreeBSD is almost working again.
Change conditional perfect to past perfect.
@rust-highfive
Copy link
Contributor

r? @Manishearth

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

@Manishearth
Copy link
Member Author

@bors r+ p=10 force

@bors
Copy link
Collaborator

bors commented Nov 17, 2015

📌 Commit e81c72e has been approved by Manishearth

bors added a commit that referenced this pull request Nov 17, 2015
@bors
Copy link
Collaborator

bors commented Nov 17, 2015

⌛ Testing commit e81c72e with merge c61e8fd...

@bors bors merged commit e81c72e into rust-lang:master Nov 17, 2015
@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.

9 participants