Skip to content

Do not fail on directories named 'opam' when scanning the packages directory for the first time - #6995

Merged
kit-ty-kate merged 2 commits into
ocaml:masterfrom
kit-ty-kate:opam-dir-repo-add
Jul 3, 2026
Merged

Do not fail on directories named 'opam' when scanning the packages directory for the first time#6995
kit-ty-kate merged 2 commits into
ocaml:masterfrom
kit-ty-kate:opam-dir-repo-add

Conversation

@kit-ty-kate

Copy link
Copy Markdown
Member

Detected in #6941

Comment thread tests/reftests/repository.test Outdated
… when faced with a repository containing an 'opam' directory
…rectory for the first time

This happens during 'opam repo add' or `opam init` but worked just fine
on subsequent 'opam update'
@kit-ty-kate
kit-ty-kate requested a review from rjbou July 3, 2026 10:22

@rjbou rjbou left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thanks!

@kit-ty-kate

Copy link
Copy Markdown
Member Author

Ignoring ocaml-benchmarks as the build machines aren't available (and opam init / opam repo add isn't benchmarked anyway)

@kit-ty-kate
kit-ty-kate merged commit 77ec804 into ocaml:master Jul 3, 2026
39 of 40 checks passed
@kit-ty-kate
kit-ty-kate deleted the opam-dir-repo-add branch July 3, 2026 11:31
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