Skip to content

Conversation

@anmonteiro
Copy link
Collaborator

@anmonteiro anmonteiro commented Jul 20, 2025

These seem to be failing with:

[WARNING] Failed checks on pupilfirst package definition from source at
git+file:///home/runner/work/dune/dune/pupilfirst#master: warning 74:
Field 'pin-depends' contains packages that are neither in 'depends' nor
in 'depopts': "reason-react-ppx"

not entirely sure that the pin-depends are being respected.

@anmonteiro anmonteiro force-pushed the anmonteiro/fix-pupilfirst-bench branch from baaf167 to ce7a030 Compare July 20, 2025 03:41
@anmonteiro
Copy link
Collaborator Author

OK it sounds like this is related to OPAM 2.4 not respecting pins for some reason when you run opam install . --deps-only.

Signed-off-by: Antonio Nuno Monteiro <[email protected]>
@anmonteiro anmonteiro force-pushed the anmonteiro/fix-pupilfirst-bench branch from e62af2d to ae0f9ac Compare July 20, 2025 18:03
@anmonteiro
Copy link
Collaborator Author

This was a regression in OPAM: ocaml/opam#6611

Thanks @kit-ty-kate for the help on this one.

@anmonteiro anmonteiro enabled auto-merge (squash) July 20, 2025 18:03
@anmonteiro anmonteiro merged commit 6059d7c into ocaml:main Jul 20, 2025
1 check passed
@anmonteiro anmonteiro deleted the anmonteiro/fix-pupilfirst-bench branch July 20, 2025 18:04
Sudha247 pushed a commit to Sudha247/dune that referenced this pull request Jul 23, 2025
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.

1 participant