Skip to content

Conversation

@gridbugs
Copy link
Contributor

No description provided.

Copy link
Collaborator

@xvw xvw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Is the version with eval (dune env) is already released?

@gridbugs
Copy link
Contributor Author

Good catch, no it's not! I didn't realise that. This command will be added in dune.3.21.0 which should be released in the next few weeks.

@gridbugs
Copy link
Contributor Author

Heads up that the documentation here for ocaml-eglot suggests using dune tools env.

@xvw
Copy link
Collaborator

xvw commented Nov 25, 2025

Heads up that the documentation here for ocaml-eglot suggests using dune tools env.

Yes but there is a clear mention of the needed version:

If you are using dune for package management on the latest nightly bulid or version 3.21

I think your instructions deserve this mention of version number too :)

to make sure `ocamllsp` can analyze the code in the project.

To make sure your editor sees the correct instance of `ocamllsp` for the
project, run `eval $(dune tools env)` from your shell before launching your
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dune tools env does not seem to work for me.

$ dune tools env
dune: unknown command 'env', must be one of 'exec', 'install' or 'which'.
Usage: dune tools COMMAND …
Try 'dune tools --help' or 'dune --help' for more information.

I'm using dune version 3.20.1

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that's the point of the following: #1573 (comment)

:)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the duplication. I just added a comment to the diff without checking if others had already complained.

@gridbugs
Copy link
Contributor Author

Updated instructions to add a minimum version number for dune.

@voodoos
Copy link
Collaborator

voodoos commented Nov 26, 2025

Thanks !

@voodoos voodoos merged commit e4e34e0 into ocaml:master Nov 26, 2025
3 of 6 checks passed
davesnx pushed a commit to davesnx/ocaml-lsp that referenced this pull request Nov 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants