Fix min and max errors on windows

[iiegor]

The latest commit that was made on peters/winpty fix this issues, so updating winpty will be fine (please remember publishing the pkg when you merge, thanks).

Fixes #131

[BenasPaulikas]

How can I make this work on my end ?

[iiegor]

@BenasPaulikas at the moment you can do npm install ptyw.js or use my fixed branch https://github.com/iiegor/pty.js/tree/win-fix

[BenasPaulikas]

On windows version .on('data') doesn't works though child_process on('data') returns all text

[iiegor]

@BenasPaulikas Strange... must work 😕 Check this, uses ptyw.js and the data event works well on windows.

[BenasPaulikas]

@iiegor Apparently launching .exe gives no data emmitter, however if I launch .bat file which launch .exe file it's working good(cmd.exe /c file.exe). Strange..

[BenasPaulikas]

@iiegor Will ptyw.js and pty.js will be merged someday ?

[iiegor]

@BenasPaulikas My apologies for closing this but this PR was opened the 10 of Sept and the author was active at that time and there wasn't any comment or interest meanwhile in merging this.

[BenasPaulikas]

@iiegor Strange, why it's being ignored? I saw lots of issues lately regarding this fix

[darkf]

@chjj I hate to be blunt like this, but are you absolutely daft? This is an essential fix and you straight up ignored it. Are you even maintaining this anymore?

It doesn't build with VS2015 either. You should think about updating winpty to a version not 3 fucking years old, maybe?

Come on, @chjj. You have time to maintain this instead of writing useless crap like browser terminals which uses this in the first place.

[BenasPaulikas]

lol

[darkf]

Nevermind, ptyw.js is an updated fork ( https://github.com/iiegor/ptyw.js )

@BenasPaulikas Looks like this will never be merged, author is too busy writing browser terminals in San Francisco. =P