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

Build fail when installing with opam #82

Closed
solvictor opened this issue Apr 13, 2022 · 7 comments
Closed

Build fail when installing with opam #82

solvictor opened this issue Apr 13, 2022 · 7 comments

Comments

@solvictor
Copy link

solvictor commented Apr 13, 2022

Windows 10 x64, Cygwin error while installing -> dune build fail
let me know if you need more informations
image

@Chimrod
Copy link

Chimrod commented May 22, 2022

Hello,

I also have the issue. The screenshot only show the last message (which are all warnings), but you can see the real cause in the log

ssl_stubs.c:880:10: error: ‘TLS1_3_VERSION’ undeclared (first use in this function); did you mean ‘TLS1_2_VERSION’?
  880 |     case TLS1_3_VERSION:
      |          ^~~~~~~~~~~~~~
      |          TLS1_2_VERSION

error.log

I do not know if this comes from an outdated package in the cygwin distribution, or a missing guard in the C stub.

@solvictor
Copy link
Author

solvictor commented May 22, 2022

Hello,

I also have the issue. The screenshot only show the last message (which are all warnings), but you can see the real cause in the log

ssl_stubs.c:880:10: error: ‘TLS1_3_VERSION’ undeclared (first use in this function); did you mean ‘TLS1_2_VERSION’?
  880 |     case TLS1_3_VERSION:
      |          ^~~~~~~~~~~~~~
      |          TLS1_2_VERSION

error.log

I do not know if this comes from an outdated package in the cygwin distribution, or a missing guard in the C stub.

I just ran it again and what you see on my screenshot is the entire ERROR I get, I don't see anything like on your screenshot on my terminal.

EDIT: i checked the log file and there is the same error as on your screen inside it

@Chimrod
Copy link

Chimrod commented May 22, 2022

This seems to be a duplicate of #70. This issue seems to be still present in v0.5.10.

@Chimrod
Copy link

Chimrod commented May 22, 2022

I’m not sure we should close this ticket : the issue #70 is already closed and the problem is still here.

note: I’m not a maintainer in the library, and I just try to analysis and understand the issue, I didn’t said in my previous message that we problem is resolved, I just wanted to point that the issue where already reported and considered as closed in november 2021. However, we are two user to still have the same problem today!

@solvictor solvictor reopened this May 23, 2022
@Chimrod
Copy link

Chimrod commented May 24, 2022

Hello,

I’ve manually patched ssl_stubs.cs as described in the pull request #73. The error is gone, and the compilation works.

But, it fails a step later, in the link stage, and this is caused by the cygwin environment: at the date of the comment, the package for mingw-openssl is still in 1.0 version, where the library libssl-devel used in the compilation is in 1.1.

Conclusion: this issue is legit, and there is a missing commit in the latest package available in opam. But solving this issue will not help to install the package in cywing, as we have another issue, but which is not related with ocaml-ssl.

@anmonteiro
Copy link
Collaborator

Does 0.5.11 work for you?

@Chimrod
Copy link

Chimrod commented Jul 24, 2022

Thanks, the last update was installed correctly!

<><> Processing actions ><><><><><><><><><><><><><><><><><><><><><>
∗ installed conf-libssl.4
∗ installed ssl.0.5.11
Done.

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

No branches or pull requests

3 participants