-
Notifications
You must be signed in to change notification settings - Fork 293
"process.getuid is not a function" on Windows #21
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
Comments
process.getuid is not a function
on Windows
Does |
I haven't yet played with docker. I'll ask around. |
Come to think of it, could you create the docker command only when the project specifies it'll be dockerizing the packages? As I'm not using docker, this will become a non-issue for me. |
Good call. I didn't realize I was creating the command for docker even if it isn't being used. Should be an easy fix, I'll do it when I get the chance. |
@ChrisAnn, you should be able to use v2.1.2 that I just pushed. I'll leave this issue open tho to track the fact that the dockerize option doesn't work on windows. |
I tried 2.1.2 and 2.3.3 and I get the error on Windows. |
@felschr are you trying to use docker? If so, do you know the answer to my above question about a |
@dschep Yes, I was. I'll check it next week. |
The help for
I ran the following command in the CMD to see what happens:
Now when I put
Maybe that's the problem? I have no clue where |
Oh, I stumbled upon the node.js documentation for |
Cool Thanks. Yeah i knew the nodeJs side of the story, was unsure of the docker side. That looks like maybe on window It could fall back to |
@dschep I tried it and this is the result:
|
Ah, that makes sense that the user wouldn't exist in the linux VM that docker for windows creates. I guess i'll just completely skip the |
process.getuid is not a function
error when running on Windows. This seems to be a POSIX only function.Are you able to conditionally exclude it?
The text was updated successfully, but these errors were encountered: