-
Notifications
You must be signed in to change notification settings - Fork 23
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
Windows #7
Comments
Hi, it should work in principle from what I know, but I don't have a working Windows-based setup to test at the moment. The error message from |
...and are the paths to reSnap and (optionally) the postprocessing script configured correctly in your plugin settings? |
Thanks for the fast reply. I was suggesting the same, and played around a bit with it. I have Windows 11 running, so Windows Defender is the antivirus on default. My vault is on a OneDrive (something like: C:\User\me\OneDrive - Work\Vault). So my hypothesis are/were:
I checked the path multiple times and tried different locations (OneDrive and locally). I am not using a post-processing atm. But maybe I have to check the Windows Defender thingy again, next week. Or finally switch to Linux... |
The path of the Vault should not be of any concern for this issue I think, since the error seems to happen before the child process (reSnap) is even launched. The only path that could be a problem is the one where reSnap lies. BTW, how are you running reSnap? It's a bash script, so you'd need to run it inside WSL, no? |
Im running reSnap using the shell script in cmd (windows command prompt). And it runs fine on its own. If I run it in the consol it works fine: However I have to put the paths in quotation marks, due to the space used in the file path. Don't know how this is handled in the plugin. |
But you are running that command
within WSL / "Bash on Ubuntu" or something like that – not within the 'normal' Windows console, right? reSnap is a bash script - as far as I'm aware, those have only become possible to run on Windows with Win10, and there also only run within WSL. I need to understand this properly to get any clue about how to make this work. I don't think the spaces are the problem, as the paths are passed as single strings (including spaces) to the appropriate node.js function. |
No, i running that in the 'normal' Windows console (Windows 11). I haven't had WSL installed on that machine yet. It also runs in the git-bash using the command 'sh'. |
The git-bash shell is not usable programmatically (to be called from node.js), so that doesn't help us. I still don't fully understand your setup, since I can't find any information on executing bash scripts under Win11 without having set up WSL at some point. Also keep in mind that That notwithstanding, my best guess is that you might be able to adapt the code of this plugin locally, changing this line: https://github.com/cobalamin/obsidian-remarkable/blob/master/main.ts#L96 from
to
or
or maybe
I hope one of these works for you. Sorry about the hassle, but I'm not able to reproduce your setup on my machine. |
Thanks! I tried some combinations of the mentioned:
Without quotation marks:
So 3. looks promising. Just |
Hmm, that's curious. Maybe for debugging, to see what's actually passed to sh, try:
|
Hard to say now at least the obsidian notice says the drawing is inserted and generates a link, but there is no .png generated nor any other information. Maybe the path for the png must be in quotation marks? I#ll try. |
I got it running... kinda. Currently a lot of hard coded paths. I had to install git and add the path to ffmpeg and lz4 to the environment paths in windows. Then I changed following part of the code:
and here:
Haven't tested it with postprocessing. |
For me this works for post-processing:
There is something strange going on with my phython installation in the moment. I think it should normally also work with |
Hi, great to hear that it's working for you now. Unfortunately I'm not sure how to make this work in a general setting, as Windows setups for this type of thing are usually messy in different ways that I can't predict. I'd be more than happy to merge a pull request, if you got this to work in a way that doesn't depend on the particularities of your Windows machine. I'd then also test on my Linux machine again to see if that still works. One common theme seems to be the need to add quotation marks, which seems odd to me and isn't necessary on my Linux machine for all I know. However I'll run some tests to see if doing so works on Linux as well. |
I can try. However I normally don't work with typescript, so this already was pushing my limits. I can try it on my other windows machine, but I think Git-bash, ffmpeg and lz4 must be installed always by hand to get it running. For the code it think it'll be the easiest to have separate functions for windows OS and linux OS and using |
Hei,
I guess it doesn't work under Windows, right? reSnap seem to work fine.
The console gives following error message:
All the best,
dermanu
The text was updated successfully, but these errors were encountered: