-
Notifications
You must be signed in to change notification settings - Fork 13
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
FR: provide OCamlPro.SuperBOL-mini #384
Comments
Putting #193 aside temporarily (might be easier to fix than it looks). I reckon we have not dug enough after #93 reached its current status (and indeed the WebAssembly solution does not seem very approachable at the moment). Yet, I think there's still a path forward available to provide a more lightweight "universal" extension via a JS-only extension. What we observed in #93 was:
Reviewing that code again, I observe an external
If that succeeds, we'd likely get an operational universal extension without much effort on the (non-LSP server) extension side. |
1 is definitely reasonable, the rest sounds good as well. ... but that belongs more to #188, this one is really about a minimal version that runs on vscode in the client parts (the view) alone - without any "vscode server" side - I think any LSP will be a server-side issue (at least if it operates on more that the current opened editor windows). The "last piece" of the need for the request of an additional "most downgraded" one is not rooted in the web-frontends (but even there a minimal version which would have a much reduced memory and cpu footprint would be useful) but in a bionic environment where I have a working REH with a system-provided node version. Opening files work, but nearly any extension that is "server side" crashes because of missing glibc. Only "client" extensions work fine. |
At the moment SuperBOL is a "workspace" extension (instead of a "UI" one that always runs "locally"). According to this documentation, providing a purely JS extension may work. Does this REH runs a VSCode server or something similar? |
and I think that's good; therefore the request for a trimmed-down version with minor adjustments to a separate UI only one (which may or may not be built from the same source tree)
yes, REH = code-server/codium-server/... which runs a node executable on a different host |
Hi Simon, could you please check the last commit on #93, and build the extension with |
Should be possible... in several days (I don't have this checked out or necessary tools installed and there's no gitpod.yml or GH codespace definition for "direct dive into" either). Please just execute those commands and upload the .vsix here (attaching a .zip, what a .vsix is after renaming should be fine), thius way I could have a quick look "soon". (and of course, #113 would help here as well) |
The file is too large to be uploaded here, I'll send it by email to you :) |
Uninstalled the previous version, then restarted VSC, then installed the one sent by mail. "locally" it runs, from Language Server Output:
Using REH does not work on all (just connected to a normally working Debian machine):
In that second environment there are no commands like "show references", but words are highlighted including hyphens, in the first environment (locally) those commands exist but don't yield any result, words are split at the hyphen. |
@emilienlemaire likely an issue in lookup of LSP server executable;
My guess here is that's more likely linked to #193. Now we know what kind of investigations the new year will start with ;) |
Wasn't the whole point of this "web" version that there shouldn't be a separate LSP server executable?!? |
No, in this "web" version the server executable is just OCaml compiled into JS (so it can run anywhere JS can run). Makes me think, a possible cause for the error is that you have a non-empty setting for |
There's only superbol-free.js in there. I did a dirty hack of adding a script there
leading to the "binary" be found and executed, but only outputting
... but the issue seems that the superbol-free "binary" is tried to be executed, while instead the node binary should be executed with a parameter to that script. [warning: I have no clue, just guessing around] |
OCamlPro.SuperBOL is nice and commonly usable.
It likely has some issues with vscode-web (see also #188), but even with a WASM server (which seems to be a long shot) it may not work in "minimal environments".
I'm currently having fun in a minimal reh environment where the remote extension host does not work.
At least in cases like this an OCamlPro.SuperBOL-mini would be nice to have a version that has no need to run there (=local editor). This would likely mean:
Can you provide an extension like this (=taking this one and deleting all parts that would need to run "wherever the code is", reducing it to the local editor? ... maybe solve #193 first as that is implicit about "using the editor window instead of files"...
This would mean at least:
The text was updated successfully, but these errors were encountered: