Skip to content

Fixes #25155 and fixes #27359 by fixing the stat defines for both fre… #27360

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 1 commit into from
Jul 29, 2015
Merged

Fixes #25155 and fixes #27359 by fixing the stat defines for both fre… #27360

merged 1 commit into from
Jul 29, 2015

Conversation

dhuseby
Copy link

@dhuseby dhuseby commented Jul 28, 2015

…ebsd 10.1 x86_64 and i686

@rust-highfive
Copy link
Contributor

r? @aturon

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

@alexcrichton
Copy link
Member

@bors: r+ d088b67

bors added a commit that referenced this pull request Jul 29, 2015
… r=alexcrichton

…ebsd 10.1 x86_64 and i686
@bors
Copy link
Collaborator

bors commented Jul 29, 2015

⌛ Testing commit d088b67 with merge 78bf4b6...

@bors bors merged commit d088b67 into rust-lang:master Jul 29, 2015
bors added a commit that referenced this pull request Jul 31, 2015
#27360 removed a padding field full of uint8_t's, but didn't remove
the use.  This didn't get picked up presumably because (a) bors
doesn't have any BSD builders, and/or (b) #[cfg]'d out blocks don't
get linted.

```
rustc: x86_64-unknown-freebsd/stage1/lib/rustlib/x86_64-unknown-freebsd/lib/liblibc
src/liblibc/lib.rs:1099:42: 1099:49 error: unused import, #[deny(unused_imports)] on by default
src/liblibc/lib.rs:1099                 use types::common::c99::{uint8_t, uint32_t, int32_t};
                                                                 ^~~~~~~
error: aborting due to previous error
fatal runtime error: Could not unwind stack, error = 159555904
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants