Skip to content

Revert "fix(ctypes): add proper user message for missing modules"#13123

Merged
Alizter merged 1 commit intoocaml:mainfrom
Alizter:push-ovxsqoxsssrl
Jan 2, 2026
Merged

Revert "fix(ctypes): add proper user message for missing modules"#13123
Alizter merged 1 commit intoocaml:mainfrom
Alizter:push-ovxsqoxsssrl

Conversation

@Alizter
Copy link
Copy Markdown
Collaborator

@Alizter Alizter commented Dec 30, 2025

This reverts commit 87faddc from #12124.

The validation introduced in #12124 is incorrect as explained in #13001. Reproduction case is here #13000. I will merge these separately so it is easier to backport.

This PR reverts the PR that introduced the incorrect validation.

@Alizter Alizter mentioned this pull request Dec 30, 2025
43 tasks
Comment thread test/blackbox-tests/test-cases/ctypes/gh12018.t Outdated
This reverts commit 87faddc.

Signed-off-by: Ali Caglayan <alizter@gmail.com>
@Alizter Alizter force-pushed the push-ovxsqoxsssrl branch from 19b5a73 to 1ee85a5 Compare January 1, 2026 23:11
@Alizter Alizter requested a review from rgrinberg January 2, 2026 09:29
@Alizter Alizter merged commit 601a0ac into ocaml:main Jan 2, 2026
30 checks passed
@Alizter Alizter deleted the push-ovxsqoxsssrl branch January 2, 2026 18:08
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