Skip to content
This repository has been archived by the owner on Apr 6, 2021. It is now read-only.

Doesn't work on windows #1

Open
jukowski opened this issue Sep 12, 2012 · 3 comments · May be fixed by #8
Open

Doesn't work on windows #1

jukowski opened this issue Sep 12, 2012 · 3 comments · May be fixed by #8

Comments

@jukowski
Copy link

Hi,

since you do the check:
if (root[0] !== "/") throw new Error("root path must start in /");
your library is not usable for windows users.

@creationix
Copy link
Contributor

Yes, there used to be a lot of other unix specific permissions bits in there. I think now that we could add windows support pretty easily though. I'll take a patch that allows windows-style absolute paths as well.

@jdinard
Copy link

jdinard commented Sep 27, 2012

I cant believe this supports Azure but it doesn't support Windows! Lol

@gjtorikian gjtorikian linked a pull request Feb 22, 2013 that will close this issue
@exsilium
Copy link

Is this project dead? Still not working on windows, I bumped into an issue similar described in this stack overflow post: http://stackoverflow.com/questions/22007738/cloud9-error-with-opening-settings-file-on-windows.

The workaround is simple enough in resolvePath function to fix the issue

    // Workaround for double root (C:\C:\) issue on Windows
    if (path.indexOf(fsOptions.root + fsOptions.root) > -1) {
      path = path.substring(fsOptions.root.length);
    }

But a larger question is, are pull requests to this project still accepted or is this repo dead/abandoned?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants