-
Notifications
You must be signed in to change notification settings - Fork 2
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
Unable to require plugin ("could not find repository at" error) #19
Comments
Can you open the debug overlay, default keybindings are Also can you paste the output of |
Thanks for your help.
On Mac:
On Windows:
|
Thanks, I can see that some of my assumptions of how the directories are created on non-linux systems are wrong. |
I'm happy to help with testing on Mac and Windows. |
I just tried on my Arch laptop and have a similar error:
|
Very strange, I would expect it to work fine on Arch, as I have tested it on NixOS... What version of wezterm are you using? Can you try to write the following in the debug overlay: |
On Arch:
On Windows:
On Mac:
|
I just tried resurrect.init_directories() -- you can delete this line once the directories have been created line from your config and check if it works? For Mac and Arch: Then checking the output of |
Ok - I've managed to get it working on Windows in a bit of a convoluted way with branch I had
If I remove either of these resurrect folders, the wezterm config crashes. But the plugin loads and works. I can save sessions. A bug I've found with windows is that it will not "resurrect" properly because path elements with spaces need to be wrapped in quotes. C:\Program Files\WindowsApps\Microsoft.PowerShell_7.4.4.0_x64__8wekyb3d8bbwe\pwsh.exe
C:\Program: The term 'C:\Program' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again. |
This is because the way Wezterm requires the plugins is by creating a directory where each
Would you expect it to be |
Yes - I think if 'Program Files' is wrapped in quotes it should work. I've got the plugin working on Arch and Mac now. I went into the plugin directory and noticed the folder wasn't initialized as a git directory. I initialized it and added |
Opened a new issue on the windows paths and closing this as it seems to be solved. Thank you @joncrangle very much for helping test it on different platforms! |
Thanks @MLFlexer If you require the plugin with: However, if you require the plugin with: I think that part of the reason I am ending up with two folders in the plugin directory is because at various points, the path construction has or doesn't have the forward slash at the end. In my config, I've noticed that state files are stored in the folder without the local plugin_name = pub.get_require_path()
assert(
plugin_name == "httpssCssZssZsgithubsDscomsZsMLFlexersZsresurrectsDswezterm",
"Inconsistent plugin name detected"
) But I think a better approach would be to handle both scenarios depending on how the user required the plugin (with or without the forward slash). |
I've been unable to add this plugin on both Mac and Windows.
On both OS, I get an error about
httpssCssZssZsgithubsDscomsZsMLFlexersZsresurrectsDswezterm
not being found or not existing.I'm using
wezterm-nightly
and requiring with:I import other plugins in my config (
smart-splits
and your wonderfulsmart_workspace_switcher.wezterm
) with no issues.The text was updated successfully, but these errors were encountered: