WSL support about uivonim HOT 13 CLOSED

Ahh right yeah its run with wsl nvim. Might see if I can find some time to look into this a little over the weekend and try get a pr up

from uivonim.

:Uivonim nc didn't seem to work, no nyan cat unfortunately 😞 any others you'd want me to try out? And it seems like completion stuff is working and tab complete is working for commands too so most of it seems to be going?

I am busy in a game-jam this weekend sorry and it feels like the issue might be a little out of scope for me considering my unfamiliarity with the projec but I am happy to keep helping you test things out on my end 🙂

from uivonim.

Can confirm thats working nicely and feels good. Just after real quick testing, it seems like my config is working as expected too. Still no nyancat tho 😛 (I didn't see any errors when trying the :Uivonim nc command tho)

from uivonim.

Great! I’ll merge that soon, and hopefully get a release up by the end of the day.

Would you mind opening an issue about nyancat not working? (Note that if you check the dev tools, you can see if there are any errors after calling :Uivonim nc which may help debug this.)

from uivonim.

@kalvinpearce I released 0.28.0 which includes the fix for this issue. If you run into any other problems, please don't hesitate to open an issue or ask about it on the Uivonim Gitter!

from uivonim.

Hello! So I’m not familiar with WSL as I don’t use Windows, but if it’s just a simple binary that can be run somehow it should be fairly straightforward. #2 would, in that case, fix the issue, as you could just specify the nvim path; however, if a specific command is required to run nvim (I.e. wsl run nvim or something along those lines), then it would require a bit more tweaking. I see no reason it couldn’t work though.

from uivonim.

@kalvinpearce I think #109 might fix this, but I'm not convinced; would you mind testing it? You don't even need to use the PR, simply change this line:

from uivonim.

Just tried it now and looks like it didn't work sorry. Double checked that the build is working and seems fine when spawning nvim but fails to spawn wsl nvim. I have added the console logs too if helps

 $> npm run start

> [email protected] start D:\Repos\uivonim
> node tools/start.js

cleaning build folder
copying index html
copying process-explorer html
copying assets
copying runtime files
running babel stuff
babel --extensions .ts,.tsx src -d build
Successfully compiled 91 files with Babel (4003ms).
electron build/bootstrap/main.js

scrdir: D:\Repos\uivonim\build
uivonim started in develop mode. you're welcome
(node:2028) electron: The default of contextIsolation is deprecated and will be changing from false to true in a future release of Electron.  See https://github.com/electron/electron/issues/23506 for more information
(node:2028) ExtensionLoadWarning: Warnings loading extension at C:\Users\Kev\AppData\Roaming\Electron\extensions\fmkadmapgofadopljbjfkapdkoienihi: Unrecognized manifest key 'browser_action'. Unrecognized manifest key 'minimum_chrome_version
'. Unrecognized manifest key 'update_url'. Cannot load extension with file or directory name _metadata. Filenames starting with "_" are reserved for use by the system.
loaded ext: React Developer Tools
(node:2028) ExtensionLoadWarning: Warnings loading extension at C:\Users\Kev\AppData\Roaming\Electron\extensions\lmhkpmbekcpmknklioeibfkpmmfibljd: Unrecognized manifest key 'commands'. Unrecognized manifest key 'homepage_url'. Unrecognized
manifest key 'page_action'. Unrecognized manifest key 'short_name'. Unrecognized manifest key 'update_url'. Permission 'notifications' is unknown or URL pattern is malformed. Permission 'contextMenus' is unknown or URL pattern is malformed.
 Permission 'tabs' is unknown or URL pattern is malformed. Cannot load extension with file or directory name _metadata. Filenames starting with "_" are reserved for use by the system.
loaded ext: Redux DevTools
reloading changes...
[2028:0129/170932.994:ERROR:CONSOLE(0)] "TypeError: Cannot read property 'isAnonymousInlineStyleSheet' of null", source:  (0)
reloading changes...
[2028:0129/170933.190:ERROR:CONSOLE(1)] "Cannot find context with specified id", source: devtools://devtools/bundled/sdk/sdk.js (1)
[2028:0129/170933.190:ERROR:CONSOLE(1)] "Extension server error: Inspector protocol error: Cannot find context with specified id", source: devtools://devtools/bundled/extensions/extensions.js (1)
[2028:0129/170933.190:ERROR:CONSOLE(1)] "Cannot find context with specified id", source: devtools://devtools/bundled/sdk/sdk.js (1)
[2028:0129/170933.190:ERROR:CONSOLE(1)] "Extension server error: Inspector protocol error: Cannot find context with specified id", source: devtools://devtools/bundled/extensions/extensions.js (1)
[2028:0129/170933.215:ERROR:CONSOLE(1)] "Connection is closed, can't dispatch pending call", source: devtools://devtools/bundled/sdk/sdk.js (1)
reloading changes...
Attempting to call a function in a renderer window that has been closed or released.
Function provided here: Object.<anonymous> (D:\Repos\uivonim\build\core\input.js:233:37
Attempting to call a function in a renderer window that has been closed or released.
Function provided here: Object.<anonymous> (D:\Repos\uivonim\build\core\input.js:233:37
Attempting to call a function in a renderer window that has been closed or released.
Function provided here: Object.<anonymous> (D:\Repos\uivonim\build\core\input.js:233:37

from uivonim.

Just tried it now and looks like it didn't work sorry. Double checked that the build is working and seems fine when spawning nvim but fails to spawn wsl nvim. I have added the console logs too if helps

Okay, no worries, I didn't think it would work. On the other hand, I believe this diff should work:

diff --git a/src/core/master-control.ts b/src/core/master-control.ts
index c878047c..3df50f73 100644
--- a/src/core/master-control.ts
+++ b/src/core/master-control.ts
@@ -56,7 +56,8 @@ const msgpackDecoder = new MsgpackStreamDecoder()
 const msgpackEncoder = new MsgpackStreamEncoder()

 const spawnVimInstance = (pipeName: string) =>
-  spawn('nvim', [
+  spawn('wsl', [
+    'nvim',
     '--cmd',
     `com! -nargs=+ -range -complete=custom,UivonimCmdCompletions Uivonim call Uivonim(<f-args>)`,
     '--embed',

That does, however, assume that any arguments passed after wsl nvim go to neovim and not to wsl. If that's not the case it would need more tweaking to get those other arguments passed to neovim. But if that works, then making it work on passing a --wsl param should be straightforward.

from uivonim.

Sorry for the late reply, but yep that dif seems to have done the trick. I haven't tested too thoroughly but its defenitely loading and reading my wsl neovim instance 🙏 thanks a bunch!

from uivonim.

Great! Just to make sure, do :Uivonim commands work and tab-complete, e.g. :Uivonim nc? I believe they should, but I can’t test that myself.

I’ll try to get a PR adding the --wsl option done and merged today (unless you’d like to implement it yourself, modeled after #109?), and also get a new release up so you can use it.

from uivonim.

:Uivonim nc didn't seem to work, no nyan cat unfortunately 😞

Hmm, it almost seems more like a Windows-specific problem than one related to WSL. Maybe the ../assets/nc.gif URL doesn't work properly on Windows?

any others you'd want me to try out?

You can just play around with :Uivonim commands if you want, like :Uivonim pick-color (which has a bug when entering the color in text, but the UI should still work), :Uivonim version (which I seem to have broken on master 😅), :Uivonim explorer, etc.

And it seems like completion stuff is working and tab complete is working for commands too so most of it seems to be going?

That's good! And yes, it does seem like most things work; WSL doesn't complicate things much, which is great. Just requires an extra prefix to the nvim command when passing --wsl. Out of curiosity, can you pass paths to wsl for the nvim binary? E.g. $ wsl /usr/bin/nvim? (Asking to see if --nvim and --wsl could/should be able to be used together.)

I am busy in a game-jam this weekend sorry and it feels like the issue might be a little out of scope for me considering my unfamiliarity with the projec but I am happy to keep helping you test things out on my end 🙂

No worries, I just wanted to offer in case you wanted to try your hand at implementing it ;) Thanks for the help testing!

from uivonim.

@kalvinpearce Would you be able to give #125 a try? The following commands should be enough to test it once you clone the branch:

# Assumes you've already run `npm install` or `npm ci`

$ npm run build
$ npx electron build/bootstrap/main.js --wsl

# Should also be able to specify path to neovim
$ npx electron build/bootstrap/main.js --wsl --nvim /path/to/nvim

from uivonim.