Skip to content

Comments

fix(opam): add pin-depends for ppx_forbid and dev-repo field#119

Merged
mathiasbourgoin merged 1 commit intomainfrom
fix/add-ppx-forbid-pin-depends
Feb 19, 2026
Merged

fix(opam): add pin-depends for ppx_forbid and dev-repo field#119
mathiasbourgoin merged 1 commit intomainfrom
fix/add-ppx-forbid-pin-depends

Conversation

@vch9
Copy link
Collaborator

@vch9 vch9 commented Feb 19, 2026

Summary

  • Add pin-depends field to miaou-core.opam to automatically pin ppx_forbid from GitHub
  • Add dev-repo field to all opam packages for better package metadata

Background

The ppx_forbid package was added as a build dependency in #118 but it's not available in the opam repository yet. This causes opam install . --deps-only --with-test to fail with:

[ERROR] Package conflict!
  * Missing dependency:
    - deps-of-miaou → deps-of-miaou-core → ppx_forbid
    unknown package

Changes

  1. miaou-core.opam: Added pin-depends field to automatically pin ppx_forbid from https://github.com/atacama-dev/ppx_forbid.git
  2. All *.opam files: Added dev-repo field pointing to the project repository

Testing

The pin-depends field works when:

  • Installing from opam repository: opam install miaou
  • Installing with opam pin: opam pin add miaou .

For local development with opam install ., users still need to manually pin:

opam pin add ppx_forbid https://github.com/atacama-dev/ppx_forbid.git -y
opam install . --deps-only --with-test -y

This is a limitation of opam's pin-depends field with local directory installs.

Note

The dev-repo field additions are standard opam package metadata that help users find the source repository.

Add pin-depends field to miaou-core.opam to automatically pin ppx_forbid
from GitHub when installing the package. This fixes dependency resolution
for ppx_forbid which is not yet available in the opam repository.

Also add dev-repo field to all opam packages for better opam metadata.

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link
Collaborator

@mathiasbourgoin mathiasbourgoin left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the nice contribution!

@mathiasbourgoin mathiasbourgoin merged commit fb17865 into main Feb 19, 2026
1 check passed
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.

2 participants