Skip to content

Commit

Permalink
compatible with rcoq9 + coq package (#57)
Browse files Browse the repository at this point in the history
* compatible with rcoq9 + coq package

* Update changelog
  • Loading branch information
thery authored Feb 7, 2025
1 parent cae8345 commit 4dd8257
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 4 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
## 0.5.0
* compatible with rcoq9 + coq package (#57 by @thery)
## 0.4.1
* Change in the xml-protocol (#811 by @thery)
## 0.4.0
Expand Down
4 changes: 2 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
"email": "[email protected]",
"url": "http://people.csail.mit.edu/cj/"
},
"version": "0.4.1",
"version": "0.5.0",
"publisher": "coq-community",
"license": "MIT",
"icon": "images/logo.png",
Expand Down
2 changes: 1 addition & 1 deletion server/src/coqtop/CoqTop.ts
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ export function detectVersion(coqtopModule: string, cwd: string, console?: {log:
});

coqtop.on('close', () => {
const ver = /^\s*The Coq Proof Assistant, version (.+?)\s/.exec(result);
const ver = /^\s*The (?:Coq Proof Assistant|Rocq Prover), version (.+?)\s/.exec(result);
// if(!ver)
// console.warn('Could not detect coqtop version');
resolve(!ver ? undefined : ver[1]);
Expand Down
4 changes: 4 additions & 0 deletions server/src/coqtop/CoqTop8.ts
Original file line number Diff line number Diff line change
Expand Up @@ -227,6 +227,10 @@ export class CoqTop extends IdeSlave8 implements coqtop.CoqTop {
}
if (semver.satisfies(this.coqtopVersion, ">= 8.9")) {
var coqtopModule = this.coqidetopBin;
if (semver.satisfies(this.coqtopVersion, ">= 9.0")) {
// with rocq not .opt
coqtopModule = coqtopModule.replace(/(.opt)$/, '');
}
// var coqtopModule = 'cmd';
var args = [
// '/D /C', this.coqPath + '/coqtop.exe',
Expand Down

0 comments on commit 4dd8257

Please sign in to comment.