Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

libc: unbreak severals os/platform #23967

Merged
merged 2 commits into from
Apr 3, 2015
Merged

Conversation

semarie
Copy link
Contributor

@semarie semarie commented Apr 2, 2015

affected struct:

  • sockaddr_storage
  • sockaddr_un

apply the same method used for linux for:

  • bitrig/openbsd
  • freebsd
  • dragonfly

this commit unbreak build for openbsd (and bitrig, freebsd and dragonfly too I think)

r? @alexcrichton

affected struct:
 - sockaddr_storage
 - sockaddr_un

apply the same method used for linux for:
 - bitrig/openbsd
 - freebsd
 - dragonfly
unbreak build for:
 - linux (mips/mipsel)
 - freebsd
 - dragonfly
 - bitrig
 - openbsd

by converting unsigned integers `-1` to `!0`
@semarie
Copy link
Contributor Author

semarie commented Apr 2, 2015

the first patch is for #23860

@semarie
Copy link
Contributor Author

semarie commented Apr 2, 2015

the second patch follow f86318d

unbreak build for:

  • linux (mips/mipsel)
  • freebsd
  • dragonfly
  • bitrig
  • openbsd

by converting unsigned integers -1 to !0

@semarie semarie changed the title libc: don't derive Clone, but impl it libc: unbreak severals os/platform Apr 2, 2015
@semarie
Copy link
Contributor Author

semarie commented Apr 2, 2015

cc @dhuseby @wg @mneumann

as I change several code that touch bitrig, freebsd or dragonfly, without be able to test it, I cc you...

@alexcrichton
Copy link
Member

@bors: r+ 26453bb

@bors
Copy link
Collaborator

bors commented Apr 2, 2015

⌛ Testing commit 26453bb with merge d17d6e7...

bors added a commit that referenced this pull request Apr 2, 2015
affected struct:
 - sockaddr_storage
 - sockaddr_un

apply the same method used for linux for:
 - bitrig/openbsd
 - freebsd
 - dragonfly

this commit unbreak build for openbsd (and bitrig, freebsd and dragonfly too I think)

r? @alexcrichton
@bors bors merged commit 26453bb into rust-lang:master Apr 3, 2015
@semarie
Copy link
Contributor Author

semarie commented Apr 3, 2015

Just as side note, the compilation for other targets than openbsd have been tested for:

  • x86_64-unknown-dragonfly
  • x86_64-unknown-freebsd
  • x86_64-unknown-bitrig
  • mips-unknown-linux-gnu
  • mipsel-unknown-linux-gnu

@wg
Copy link
Contributor

wg commented Apr 3, 2015

Thank you @semarie! The test suite is still running on my FreeBSD 10.1 system but I see the build bot passed. Thank you for fixing so many of the recent *BSD breakages!

@semarie semarie deleted the libc-clone branch April 3, 2015 04:10
@semarie
Copy link
Contributor Author

semarie commented Apr 5, 2015

@alexcrichton these commits are needed for beta under openbsd (and all others targets affected by this problem). Is it possible to import them to beta ?

@alexcrichton
Copy link
Member

We're likely to update the beta branch to the master branch for the next week or two as some final changes trickle in, so this should hopefully get picked up soon!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants