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

Fix most libc-test compile warnings and errors for DragonFly #2461

Merged
merged 2 commits into from
Oct 17, 2021

Conversation

rtzoeller
Copy link
Contributor

libc-test is failing on my DragonFly BSD system with a large number of errors, most of which are just a result of headers not being included.

[root@ ~/libc/libc-test]# uname -a
DragonFly  6.0-RELEASE DragonFly v6.0.1-RELEASE #0: Wed Oct 13 21:06:17 CDT 2021     root@:/usr/obj/usr/src/sys/X86_64_GENERIC  x86_64

[root@ ~/libc/libc-test]# cargo test 2>&1 >/dev/null | wc -l
    6015

Including a few additional headers, most of these warnings and errors are fixed:

[root@ ~/libc/libc-test]# cargo test 2>&1 >/dev/null | wc -l
    561

@rust-highfive
Copy link

r? @JohnTitor

(rust-highfive has picked a reviewer for you, use r? to override)

Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Unfortunately, we don't have CI for DragonFlyBSD and it's hard to find currently :(

@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Oct 17, 2021

📌 Commit 0c4b6ed has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Oct 17, 2021

⌛ Testing commit 0c4b6ed with merge 4aaba6c...

@bors
Copy link
Contributor

bors commented Oct 17, 2021

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing 4aaba6c to master...

@bors bors merged commit 4aaba6c into rust-lang:master Oct 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants