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

Deprecate use_std cargo feature: use std instead . #1361

Merged
merged 1 commit into from
May 24, 2019

Conversation

gnzlbg
Copy link
Contributor

@gnzlbg gnzlbg commented May 24, 2019

Related to #657 .

@rust-highfive
Copy link

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

@gnzlbg
Copy link
Contributor Author

gnzlbg commented May 24, 2019

cc @dtolnay this essentially fixes #657 , once we release the next breaking version of libc, we can remove the use_std feature, completing the fix.

@gnzlbg gnzlbg requested a review from alexcrichton May 24, 2019 12:17
@gnzlbg
Copy link
Contributor Author

gnzlbg commented May 24, 2019

@bors: r+

@bors
Copy link
Contributor

bors commented May 24, 2019

📌 Commit 4ac26af has been approved by gnzlbg

@bors
Copy link
Contributor

bors commented May 24, 2019

⌛ Testing commit 4ac26af with merge bb41337...

bors added a commit that referenced this pull request May 24, 2019
Deprecate `use_std` cargo feature: use `std` instead .

Related to #657 .
@bors
Copy link
Contributor

bors commented May 24, 2019

💔 Test failed - checks-travis

@gnzlbg
Copy link
Contributor Author

gnzlbg commented May 24, 2019

@bors: retry

bors added a commit that referenced this pull request May 24, 2019
Deprecate `use_std` cargo feature: use `std` instead .

Related to #657 .
@bors
Copy link
Contributor

bors commented May 24, 2019

⌛ Testing commit 4ac26af with merge b722d8c...

@bors
Copy link
Contributor

bors commented May 24, 2019

☀️ Test successful - checks-cirrus, checks-travis, status-appveyor
Approved by: gnzlbg
Pushing b722d8c to master...

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.

4 participants