-
Notifications
You must be signed in to change notification settings - Fork 12
Consider supporting Windows #15
Comments
This is a good idea! Unfortunately, I don't know anything about windows and don't have easy access to a system I can test on. I'll definitely need help on this one. What about blasting away You might get those same permission denied errors from before, however. This would only help if winfsp sets it to defaults that work. |
You mean like not even passing them to the IIRC I also read about a flag in winfsp for setting default permissions |
@Cobertos did you get a chance to test removing |
Nope, not yet. I've been buried in other projects but I should have some
more time tonight.
…On Fri, Feb 22, 2019, 12:45 AM Tanner Collin ***@***.***> wrote:
@Cobertos <https://github.com/Cobertos> did you get a chance to test
removing self.uid and self.gid?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#15 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AGcjCFn078JWvQzbsI2r9p-t2QcG4rlKks5vP4QRgaJpZM4bALPM>
.
|
Tried blowing away the passing of |
So after looking at this more, I think the route needs to be either to add all the windows functions from win32api or just I figured out how to use |
Sounds good, thanks for looking at this! |
I would also be really interested in Windows support. @Cobertos do you have any thoughts on where you left off with this? |
IIRC, using Cloning I've since transitioned to using a different note app though, so I haven't put any more work into this. |
@Cobertos which note app? I'm always interested in alternatives. Edit: I see you're using Notion.so. What made you switch? Do you pay for it? |
Notion.so, not selfhosted or e2e encrypted, but it does a lot of stuff
right and the API has been a dream. The nested note structure just really
works for my brain and the data model backing it is super easy to work with.
…On Wed, Apr 22, 2020, 11:10 PM Tanner Collin ***@***.***> wrote:
@Cobertos <https://github.com/Cobertos> which note app? I'm always
interested in alternatives.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#15 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABTSGCAI7JP7Z6XUAZ6AMMLRN6WSZANCNFSM4GYAWPGA>
.
|
Your upstream
fusepy
supports Windows usingwinfsp
.I was able to get
standardnotes-fs
to run on Windows but wasn't able to get it to actually work. Because I was originally using dummy gid and uid values, I would get permission denied errors (similar to this user doing a similar migration)It seems the only change that needs to be made is to have Windows call it's equivalent of
getuid()
andgetgid()
and then register the conversion from Windows sid to uid/gid with winfsp. I tried my best but got stuck when trying to convert between thepywin32
sid object and thewinfspy
cffi
binding sid object. They're not the same size and the latter doesn't expose anything do me to use and I'm not familiar enough with the intricacies of these frameworks to get further.Here's what I ended up with, replaced a few lines in
sn_fuse.py
The text was updated successfully, but these errors were encountered: