Skip to content
New issue

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

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

Already on GitHub? # to your account

Use enabled_if to control the Windows stubs #4777

Closed
wants to merge 1 commit into from

Conversation

dra27
Copy link
Member

@dra27 dra27 commented Jul 29, 2021

Still draft as I need to check in more detail whether some of the compiler tests in configure either need to be removed entirely or converted to be called from dune.

@rjbou rjbou added this to the 2.2.0~alpha milestone Nov 16, 2021
@kit-ty-kate kit-ty-kate added the PR: WIP Not for merge at this stage label Jan 18, 2022
@dra27 dra27 removed this from the 2.2.0~alpha milestone May 17, 2022
@rjbou rjbou force-pushed the dune-enabled-if branch from 70c1893 to 2bddd9d Compare July 12, 2023 11:05
@rjbou
Copy link
Collaborator

rjbou commented Sep 13, 2023

rebased

@dra27
Copy link
Member Author

dra27 commented Sep 17, 2024

Fixed in #5381

@dra27 dra27 closed this Sep 17, 2024
@dra27 dra27 deleted the dune-enabled-if branch September 17, 2024 16:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
PR: WIP Not for merge at this stage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong stubs are used when opam sources are embedded in a dune workspace
3 participants