-
Notifications
You must be signed in to change notification settings - Fork 74
Support Windows #72
Comments
+1 |
+1 |
+1 |
I have not any machine on Windows on my side, so if someone want to add the Windows support it's welcome. |
@tamtakoe you can try to use it under cygwin |
hey guys, What are the issues that everyone's seeing? I submitted a few PRs and fixed any Windows related bugs I've seen. |
@AntJanus for example, the problems that are described in the topic :-) |
@AntJanus: How do you use it on windows ? Shell commands are hardcoded in the code like "rm -rf ...". Maybe you are using cygwin or equivalent. @neoziro: I think that's a conception issue. Not having windows is not blocking. Node being portable, that kind of operation should be done in pure node and not with shell commands. At least, you should write it in the readme to avoid people wasting time testing something that will never work. |
@jdat82 ahhh, could be! I'm using CMDer which has a bunch of GNU tools loaded with it. I could help make it more portable by replacing @neoziro I can definitely try to make |
I close it, if you can run it on windows @AntJanus it's OK, and I agree, feel free to create a module specially designed for Windows. |
Deploy doesn't work in Windows. Problem with paths to create an archive (extra slash at the beginning of the path. Problem in
path2
module). It doesn't work in PowerShell (it doesn't support&&
command separator, only;
). Problem with SCP command.The text was updated successfully, but these errors were encountered: