Misinterpretation of META's exists_if directive by dune.
#10563
Comments
dbuenzli
changed the title
META's exist_if directive by dune.META's exist_if directive by dune.
dbuenzli
changed the title
META's exist_if directive by dune.META's exists_if directive by dune.
|
Good detective work; fix as suggested in #10564. |
emillon
added a commit
to emillon/opam-repository
that referenced
this issue
May 24, 2024
CHANGES: ### Fixed - Fix interpretation of `exists_if` predicate in `META` files of installed libraries containing more than one element. (ocaml/dune#10564, fixes ocaml/dune#10563, @dbuenzli, @nojb) - Fix TSAN warning in wait4 stubs (ocaml/dune#10554, fixes ocaml/dune#10553, @emillon)
avsm
pushed a commit
to avsm/opam-repository
that referenced
this issue
Sep 5, 2024
CHANGES: ### Fixed - Fix interpretation of `exists_if` predicate in `META` files of installed libraries containing more than one element. (ocaml/dune#10564, fixes ocaml/dune#10563, @dbuenzli, @nojb) - Fix TSAN warning in wait4 stubs (ocaml/dune#10554, fixes ocaml/dune#10553, @emillon)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It seems that
dunemisinterprets the semantics ofMETA'sexists_iffield.The package should exist if any of the file is exists (see this page), not all of them. Basically this line should be turned into an
List.exists.This hides the latest cmdliner (1.3.0) release on 32-bit platforms, see ocaml/opam-repository#25924
The text was updated successfully, but these errors were encountered: