Skip to content

Fix typo in the coq-mathcomp-dioid compatibility shim - #2

Merged
proux01 merged 1 commit into
math-comp:masterfrom
JasonGross:fix-rocq-mathcomp-dioid-typo
Aug 31, 2026
Merged

Fix typo in the coq-mathcomp-dioid compatibility shim#2
proux01 merged 1 commit into
math-comp:masterfrom
JasonGross:fix-rocq-mathcomp-dioid-typo

Conversation

@JasonGross

Copy link
Copy Markdown
Contributor

coq-mathcomp-dioid depends on rocq-mathcomp-diod, which does not exist — the package this repo defines is rocq-mathcomp-dioid.

The effect is that coq-mathcomp-dioid cannot be satisfied at all. opam's only way to resolve a switch containing it is to remove it:

=== remove 2 packages
  ⊘ coq-mathcomp-dioid   dev (pinned)   [conflicts with coq-core]

Introduced in 1946231 (#1), so it affects master only.


Opened autonomously by Claude (Opus 5) on behalf of Jason Gross (jason@theorem.dev).

The shim depended on rocq-mathcomp-diod, which does not exist, so
coq-mathcomp-dioid was unsatisfiable: opam's only way to resolve it was
to remove the package. Introduced in 1946231 (math-comp#1).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@proux01
proux01 merged commit 40db610 into math-comp:master Aug 31, 2026
1 check passed
@proux01

proux01 commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Thanks
Funny that Claude still managed to write six lines for that ;). These are really verbose.

@JasonGross

Copy link
Copy Markdown
Contributor Author

They really are

@JasonGross
JasonGross deleted the fix-rocq-mathcomp-dioid-typo branch August 31, 2026 14:48
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