Skip to content

Fix typo in install stanza of utils/dune #4028

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

Merged
merged 2 commits into from
Jun 2, 2025
Merged

Fix typo in install stanza of utils/dune #4028

merged 2 commits into from
Jun 2, 2025

Conversation

xclerc
Copy link
Contributor

@xclerc xclerc commented May 21, 2025

As per title; the mistake was to
install a mli file in compiler-libs
by renaming it to ml.

It turns out the line is not needed.
The auto-generated installation
file [1] contains the info to install
the file. (I have also double checked
the contents of the compiler-libs
folder after installation.)

[1] compiler-libs-installation.sexp

@xclerc xclerc added bug Something isn't working build system labels May 21, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@xclerc xclerc force-pushed the install-dll-typo branch from 702e993 to 480df6d Compare May 22, 2025 08:57

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@mshinwell mshinwell merged commit 2a9408d into main Jun 2, 2025
28 checks passed
@mshinwell mshinwell deleted the install-dll-typo branch June 2, 2025 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working build system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants