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

ssl: use github mirror for tarball #4477

Merged
merged 2 commits into from
Jul 17, 2015
Merged

ssl: use github mirror for tarball #4477

merged 2 commits into from
Jul 17, 2015

Conversation

djs55
Copy link
Contributor

@djs55 djs55 commented Jul 17, 2015

This works-around the current sourceforge outtage. Unfortunately both
the URL and md5 change.

Fixes #4476

Signed-off-by: David Scott dave@recoil.org

This works-around the current sourceforge outtage. Unfortunately both
the URL and md5 change.

Fixes ocaml#4476

Signed-off-by: David Scott <dave@recoil.org>
@djs55
Copy link
Contributor Author

djs55 commented Jul 17, 2015

hm, this doesn't work because the tagged version tarball doesn't contain files like 'configure'

Signed-off-by: David Scott <dave@recoil.org>
whitequark added a commit that referenced this pull request Jul 17, 2015
ssl: use github mirror for tarball
@whitequark whitequark merged commit 42f35dd into ocaml:master Jul 17, 2015
@dsheets
Copy link
Member

dsheets commented Jul 21, 2015

Wow almost 6000 lines included in the opam repository! Does upstream know about these problems (sourceforge and archives without autoconf outputs)? It would be nice to not distribute 168K of configure script to every opam repository cloner unbidden for every future version of the SSL bindings...

@djs55
Copy link
Contributor Author

djs55 commented Jul 21, 2015

I agree that this is an ugly workaround -- I've created an issue over at savonet/ocaml-ssl to see what they think.

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.

None yet

3 participants