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

Use `#[link(cfg(..))]` in preparation for libstd #446

Merged
merged 1 commit into from Nov 10, 2016

Conversation

Projects
None yet
4 participants
@alexcrichton
Copy link
Member

alexcrichton commented Nov 10, 2016

In preparation for rust-lang/rust#37545 this is adding the appropriate
directives to libc to get included.

Use `#[link(cfg(..))]` in preparation for libstd
In preparation for rust-lang/rust#37545 this is adding the appropriate
directives to libc to get included.
@rust-highfive

This comment has been minimized.

Copy link

rust-highfive commented Nov 10, 2016

@alexcrichton: no appropriate reviewer found, use r? to override

@alexcrichton

This comment has been minimized.

Copy link
Member Author

alexcrichton commented Nov 10, 2016

@bors: r+

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Nov 10, 2016

📌 Commit f9323d1 has been approved by alexcrichton

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Nov 10, 2016

⌛️ Testing commit f9323d1 with merge 106fbed...

bors added a commit that referenced this pull request Nov 10, 2016

Auto merge of #446 - alexcrichton:link-cfg, r=alexcrichton
Use `#[link(cfg(..))]` in preparation for libstd

In preparation for rust-lang/rust#37545 this is adding the appropriate
directives to libc to get included.

@alexcrichton alexcrichton merged commit 6e8c1b4 into rust-lang:master Nov 10, 2016

2 of 3 checks passed

homu Testing commit f9323d1 with merge 106fbed...
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@alexcrichton alexcrichton deleted the alexcrichton:link-cfg branch Nov 10, 2016

@joerg-krause

This comment has been minimized.

Copy link
Contributor

joerg-krause commented on src/unix/mod.rs in f9323d1 Nov 29, 2016

This should not be architecture dependent. This way, the crt-static feature does not work with my custom ARMv5TE musl target.

Susurrus pushed a commit to Susurrus/libc that referenced this pull request Mar 26, 2017

Auto merge of rust-lang#446 - kamalmarhubi:bind-mount-in-ci, r=posborne
mount: Run test_mount_bind in CI

Looks like Travis allows bind mounts in their container infra now.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.