Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Rename package to old-http, bump version minor, and use crates.io. #185

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

DiamondLovesYou
Copy link

No description provided.

@chris-morgan
Copy link
Owner

I explicitly did not put it on crates.io and would strongly prefer it not
to be put on crates.io and that any instances of it be removed. It’s
obsolete and people should no longer be using it. It's already
Cargo-compatible, so the only reason for it to be on crates.io is so that
other crates.io packages can depend on it, something that people should not
be doing. They should probably use Hyper instead.

This library is obsolete. Please let it die.

@Ogeon
Copy link
Collaborator

Ogeon commented Dec 6, 2014

I have to agree with you on that point. Every convenience addition to this library will extend its lifetime and make it more rooted in the ecosystem. I could agree to replace the dependencies to make the transition to something else a little smoother, but not replacing them may motivate people to finally abandon it. I do, on the other hand, think that suddenly rendering it unusable may be a bit harsh. I don't know how much it clashes at the moment. It may be acceptable for an obsolete library on its way to the graveyard.

@reem
Copy link
Collaborator

reem commented Dec 7, 2014

I think the hardest-hit targets here would be nickel, since they build on rust-http. I managed to port Iron over to Hyper, but it took quite a bit of time and there was a lot of breakage that then coincided with the other change of moving to crates.io.

@Ogeon
Copy link
Collaborator

Ogeon commented Dec 7, 2014

The change itself will not be perfectly smooth, but having two usable
libraries will at least let depending projects change when there's time.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants