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

[new release] x509 (0.16.5) #24045

Merged
merged 1 commit into from
Jul 4, 2023
Merged

Conversation

hannesm
Copy link
Member

@hannesm hannesm commented Jul 3, 2023

Public Key Infrastructure (RFC 5280, PKCS) purely in OCaml

CHANGES:

CHANGES:

* Always embed local_key_id in PKCS12 bags (reported mirleft/ocaml-x509#163 by @NightBlues,
  revised and implemented by @hannesm)
@hannesm
Copy link
Member Author

hannesm commented Jul 3, 2023

macos tests: I suspect these are still related to the miscompilation of some EC code (of mirage-crypto-ec) in some llvm version on arm64 macos... maybe you could update to the latest llvm there (see mit-plv/fiat-crypto#1606 (comment) and the comments up and down thereof)

@hannesm
Copy link
Member Author

hannesm commented Jul 4, 2023

CI failures are unrelated -- either Cancelled (timeout) or OpamSolution.Fetch_fail("Sys_error(\"No space left on device\")") -- so imho good to merge.

@avsm
Copy link
Member

avsm commented Jul 4, 2023

Regarding the macOS compilation, could the host compiler perhaps be looked at @mtelvers @tmcgilchrist? #24045 (comment)

Agreed rest looks good, merging!

@avsm avsm merged commit 3befa84 into ocaml:master Jul 4, 2023
@hannesm hannesm deleted the release-x509-v0.16.5 branch July 4, 2023 14:06
@tmcgilchrist
Copy link
Contributor

@avsm the compiler version we are running is clang-1403.0.22.14.1, which is the latest Apple provide for this version of macOS. I don't think we should be upgrading beyond that since that is the vendor supported compiler and what a user would get by default. If this package needs an alternate compiler to work on macOS-arm64 then it should be a depext on the homebrew version required.

$ sw_vers 
ProductName:		macOS
ProductVersion:		13.4.1
BuildVersion:		22F82
$ clang --version
Apple clang version 14.0.3 (clang-1403.0.22.14.1)
Target: arm64-apple-darwin22.5.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

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