Skip to content

Conversation

UnixJunkie
Copy link
Contributor

dokeysto, dokeysto_camltc, dokeysto_lz4

Any software relying on dokeysto should use this new version (4.0.2).

Really stupid use of Unix.read somewhere replaced by a really_read function (like Stdlib.really_input but for Unix file descriptors).

dokeysto, dokeysto_camltc, dokeysto_lz4
Copy link
Member

@jmid jmid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

CI is failing

  • on opam-2.0 (these can be ignored)
  • on Fedora-42 (which looks like an internal CI issue)
  • on x86_32-ocaml-4.14 running the test suite for dokeysto_camltc with Fatal error: exception Failure("stat error"). The package installs cleanly though.
  • on Windows compiling dokeysto_camltc
  • on Windows installing mingw-w64-shims.0.2.0

Potentially we could add

  • available: os != "win32" for dokeysto_camltc and
  • guard with-test for dokeysto_camltc with arch != "x86_32"

Finally: Would you consider adding x-maintenance-intent entries for these packages?
https://github.com/ocaml/opam-repository/blob/master/governance/policies/archiving.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants