Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Future bug report: select is confused by bigarray on ocaml 5.1.1 #9981

Closed
wants to merge 2 commits into from

Conversation

ElectreAAS
Copy link
Collaborator

This PR includes #9980, but the interesting part is the second commit.
The difference between test/.../c/dune and test/.../d/dune is only the order of the branches of the select. That should not change anything, and yet it does.
bigarray is both available and not available.

If we find a fix that would solve item 13 in #9954.

Signed-off-by: Ambre Austen Suhamy <ambre@tarides.com>
Signed-off-by: Ambre Austen Suhamy <ambre@tarides.com>
@ElectreAAS ElectreAAS changed the title Future bug report: select is confused by bigarray Future bug report: select is confused by bigarray on ocaml 5.1.1 Feb 12, 2024
moyodiallo added a commit to moyodiallo/dune that referenced this pull request Feb 13, 2024
This is related to ocaml#9981.

Signed-off-by: Alpha DIALLO <moyodiallo@gmail.com>
moyodiallo added a commit to moyodiallo/dune that referenced this pull request Feb 13, 2024
This is related to ocaml#9981.

Signed-off-by: Alpha DIALLO <moyodiallo@gmail.com>
moyodiallo added a commit to moyodiallo/dune that referenced this pull request Feb 13, 2024
This is related to ocaml#9981.

Signed-off-by: Alpha DIALLO <moyodiallo@gmail.com>
@ElectreAAS
Copy link
Collaborator Author

Fixed in #10011.

@ElectreAAS ElectreAAS closed this Feb 23, 2024
@ElectreAAS ElectreAAS deleted the select-bug branch February 23, 2024 15:19
rgrinberg pushed a commit to moyodiallo/dune that referenced this pull request Mar 6, 2024
This is related to ocaml#9981.

Signed-off-by: Alpha DIALLO <moyodiallo@gmail.com>
rgrinberg pushed a commit that referenced this pull request Mar 6, 2024
This is related to #9981.

Signed-off-by: Alpha DIALLO <moyodiallo@gmail.com>
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