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

Run vscoq.path as a shell command from the project directory #715

Merged
merged 1 commit into from
Jan 30, 2024

Conversation

afdw
Copy link
Contributor

@afdw afdw commented Jan 26, 2024

This fixes an inconsistency between using child_process.exec, which runs the provided string as a shell command, and vscode-languageclient, which by default did not.

Also, vscoqtop is now always run from the directory of the project.

All that allows to use VsCoq for Coq projects built with dune, by putting the following to .vscode/settings.json (where theories/Theory.v is some Coq file):

{
    "vscoq.path": "dune coq top --toplevel vscoqtop theories/Theory.v --"
}

This fixes an inconsistency between using `child_process.exec`, which runs the provided string as a shell command, and `vscode-languageclient`, which by default did not.

Also, `vscoqtop` is now always run from the directory of the project.

All that allows to use VsCoq for Coq projects built with dune, by putting the following to `.vscode/settings.json` (where `theories/Theory.v` is some Coq file):
```json
{
    "vscoq.path": "dune coq top --toplevel vscoqtop theories/Theory.v --"
}
```
@rtetley
Copy link
Collaborator

rtetley commented Jan 30, 2024

Merging as CI issues have been resolved in main. Thanks for your contribution !

@rtetley rtetley merged commit a972953 into coq:main Jan 30, 2024
7 of 9 checks passed
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