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

Revert deprecation of IpAddr, stabilizing for 1.7 #31438

Merged
merged 1 commit into from Feb 10, 2016

Conversation

Projects
None yet
4 participants
@aturon
Copy link
Member

aturon commented Feb 6, 2016

After considerable pushback, it's clear that there is a community consensus around providing IpAddr in the standard library, together with other APIs using it.

This commit reverts from deprecated status directly to stable. The deprecation landed in 1.6, which has already been released, so the stabilization is marked for 1.7 (currently in beta; will require a backport).

r? @alexcrichton

@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented Feb 6, 2016

r=me, but looks like the build failure on travis is legit (needs a stability attribute for the fields of the enum variants)

Revert deprecation of IpAddr, stabilizing for 1.7
After [considerable
pushback](rust-lang/rfcs#1451), it's clear
that there is a community consensus around providing `IpAddr` in the
standard library, together with other APIs using it.

This commit reverts from deprecated status directly to stable. The
deprecation landed in 1.6, which has already been released, so the
stabilization is marked for 1.7 (currently in beta; will require a backport).

@aturon aturon force-pushed the aturon:stab-ip-addr branch from 2da8b06 to 2067c57 Feb 9, 2016

@aturon

This comment has been minimized.

Copy link
Member Author

aturon commented Feb 9, 2016

@bors: r=alexcrichton

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Feb 9, 2016

📌 Commit 2067c57 has been approved by alexcrichton

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Feb 10, 2016

⌛️ Testing commit 2067c57 with merge 0542745...

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

Auto merge of #31438 - aturon:stab-ip-addr, r=alexcrichton
After [considerable pushback](rust-lang/rfcs#1451), it's clear that there is a community consensus around providing `IpAddr` in the standard library, together with other APIs using it.

This commit reverts from deprecated status directly to stable. The deprecation landed in 1.6, which has already been released, so the stabilization is marked for 1.7 (currently in beta; will require a backport).

r? @alexcrichton

@bors bors merged commit 2067c57 into rust-lang:master Feb 10, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details

@alexcrichton alexcrichton added the T-libs label Feb 10, 2016

@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented Feb 11, 2016

Accepting for a beta backport (discussed at libs triage today)

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.