laktek / punch

A fun and easy way to build modern websites
http://laktek.github.com/punch
MIT License
1.17k stars 107 forks source link

Windows installation fails due to pty.js dependency #38

Closed bkize closed 11 years ago

bkize commented 11 years ago

The punch dependency on sftp introduces another dependency on pty.js. When a node-gyp rebuild happens during node install punch, the following error occurs:

 ..\src\pty.cc(20): fatal error C1083: Cannot open include file: 'unistd.h': No such file or directory [C:\dev\punch\node_modules\punch\node_modules\sftp\node_modules\pty.js\build\pty.vcxproj]

The lack of unistd.h on Windows is documented here.

This closed pty.js issue includes a statement by the author of pty.js regarding support for Windows, which can be summarized as "not possible".

Here are details regarding my configuration:

I hope I'm overlooking an obvious solution or workaround as I was looking forward to trying Punch.

Brian

laktek commented 11 years ago

Unfortunately, I haven't got access to a Windows 7 machine to see the exact issue. But if dependencies for SFTP is causing the issue, we can consider to move it from the core and make it a plugin.

laktek commented 11 years ago

Try installing the latest version of Punch - 0.5.0. I have moved SFTP publishing has been moved to its own plugin. This will also remove the problematic dependency on pty.js.

bkize commented 11 years ago

Yes, the new Punch version fixed the issue. Punch installed without an error on this Windows 7 system. Thanks for the prompt response and update.

Brian