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

Migrate to rebar3 and use hackney from hex.pm #42

Merged
merged 2 commits into from
Aug 7, 2017

Conversation

tverlaan
Copy link
Contributor

@tverlaan tverlaan commented Jul 28, 2017

I added a rebar.lock as well as created by rebar3. As discussed in #41

@tverlaan
Copy link
Contributor Author

Is supporting R17 a requirement? I'm not sure what the difference is in deps by using rebar vs rebar3.

@tverlaan tverlaan changed the title Migrate to rebar3 and use hackney from hex.pm GH#41 Migrate to rebar3 and use hackney from hex.pm #41 Jul 28, 2017
@tverlaan tverlaan changed the title Migrate to rebar3 and use hackney from hex.pm #41 Migrate to rebar3 and use hackney from hex.pm Jul 28, 2017
@surik
Copy link
Contributor

surik commented Jul 28, 2017

Looks erlang-idna needs OTP >= 18 https://github.com/benoitc/erlang-idna/blob/master/rebar.config#L5. Which is weird because hackney says:
Supported versions of Erlang are R16B03-1, 17.3.4 and above. It is reported to work with R14B04 and R15B03-1.

@tverlaan
Copy link
Contributor Author

The next hackney release (> 1.8.6) will depend on idna 5.1.0 which doesn't contain minimum OTP version. However next mayor version will only support Erlang >= 19 as per benoitc/hackney#425.

@surik
Copy link
Contributor

surik commented Jul 31, 2017

Then can you update hackney to have idna 5.1.0?

@tverlaan
Copy link
Contributor Author

tverlaan commented Aug 7, 2017

Can we now merge this?

@surik surik merged commit f484d1e into travelping:master Aug 7, 2017
@surik
Copy link
Contributor

surik commented Aug 7, 2017

@tverlaan sorry for delay. thank you.

@tverlaan tverlaan deleted the rebar3_with_hex branch August 7, 2017 10:45
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.

2 participants