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

coq 8.9.0 support #75

Open
VisualizeTheWorld opened this issue Feb 22, 2019 · 4 comments · May be fixed by #76
Open

coq 8.9.0 support #75

VisualizeTheWorld opened this issue Feb 22, 2019 · 4 comments · May be fixed by #76

Comments

@VisualizeTheWorld
Copy link

When issuing :CoqLaunch under Coq 8.9.0, vim freezes with the message

Don't know what to do with -ideslave
See -help for the list of supported options

vim does not respond to ^c or ^z after this point, and must be killed.

@splintersuidman
Copy link

The changelog for Coq 8.9.0 states:

Coq binaries and process model

  • Before 8.9, Coq distributed a single coqtop binary and a set of dynamically loadable plugins that used to take over the main loop for tasks such as IDE language server or parallel proof checking.

    These plugins have been turned into full-fledged binaries so each different process has associated a particular binary now, in particular coqidetop is the CoqIDE language server, and coq{proof,tactic,query}worker are in charge of task-specific and parallel proof checking.

Instead of coqtop -ideslave, coqidetop should be run. The fix is the following diff:

diff --git a/autoload/coqtop.py b/autoload/coqtop.py
index 3a181d0..0dc5733 100644
--- a/autoload/coqtop.py
+++ b/autoload/coqtop.py
@@ -203,8 +203,7 @@ def send_cmd(cmd):
 def restart_coq(*args):
     global coqtop, root_state, state_id
     if coqtop: kill_coqtop()
-    options = [ 'coqtop'
-              , '-ideslave'
+    options = [ 'coqidetop'
               , '-main-channel'
               , 'stdfds'
               , '-async-proofs'

I could open a pull request with this change, but I'm not sure how to deal with backwards-compatibility. (And this repository does not seem to be very active.)

@splintersuidman
Copy link

I have created a fork at https://github.com/splintah/coquille. In the mean time, you can use the plugin from there.

@VisualizeTheWorld
Copy link
Author

Thanks!

@coord-e coord-e linked a pull request Mar 21, 2019 that will close this issue
@code-hunger
Copy link

I came here from a Google search, maybe it's worth mentioning this in the README.

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 a pull request may close this issue.

3 participants