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

hex_api: transition away from http_uri #82

Merged
merged 3 commits into from
Jan 14, 2020

Conversation

michaelklishin
Copy link
Contributor

which has been deprecated in OTP 23 [master].

See erlang/rebar32195 and erlang/rebar3#2191 for the background.

I would be happy to move the to_list/1 function to a utility module but there doesn't seem to be one at the moment. Rebar3 already has one, so I'm not sure if it's worth introducing a new module here.

which has been deprecated in OTP 23 [master].

See erlang/rebar32195 and erlang/rebar3#2191 for
background.
@michaelklishin
Copy link
Contributor Author

Worth mentioning that this assumes that we have to support OTP versions < 21 because, well, Rebar3 still supports OTP 18.

If that's not the case this code can be simplified.

@michaelklishin
Copy link
Contributor Author

Any feedback on this?

@starbelly
Copy link
Contributor

Not a hex team member, but I can say it should support all the way back to 18. If it does not then we have to do hacks in rebar3.

@wojtekmach
Copy link
Member

I'll review this more closely this week but I think this is the way to go, thank you for the PR. And yes, we need to support old OTP versions, down to OTP 17 in fact.

@michaelklishin
Copy link
Contributor Author

Just curious, why is the obligation for support of those four-and five-year-old Erlang versions? The OTP team doesn't update even OTP 19 any more. Is there a support policy that demonstrates when support for Erlang < 21 will be dropped?

@wojtekmach
Copy link
Member

hex_core is used in hex and hex is compatible with elixir 1.x, 1.0 requires otp 17. There will definitely be future releases of hex and hex_core that will drop compatibility with old versions but nothing set in stone yet.

@michaelklishin
Copy link
Contributor Author

Any feedback on this? Warnings in Rebar currently make it impossible to build it on OTP 23. We have a Rebar PR ready and waiting for this module to be upstreamed.

@wojtekmach
Copy link
Member

@michaelklishin LGTM. I made very minor changes, just moving some code around to keep the old supporting more tied together, let me know what you think. And sorry for the delay.

@michaelklishin
Copy link
Contributor Author

@wojtekmach sure, LGTM. Thanks.

@wojtekmach wojtekmach merged commit aa33cc8 into hexpm:master Jan 14, 2020
@michaelklishin michaelklishin deleted the otp-23-master-compat branch January 24, 2020 05:18
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.

3 participants