Windows suppport

[jsep]

No description provided.

[serut]

Nice !

[serut]

Still some issue on windows 10

Native thread-sleep not available.

D:\dev\meteor-app>spacejam test-packages
Native thread-sleep not available.
This will result in much slower performance, but it will still work.
You should re-install spawn-sync or upgrade to the lastest version of node if possible.

It can be fixed it by adding "try-thread-sleep": "1.0.0", to the package.json of spacejam
(source: google/web-starter-kit#691).
I also updated cross-spawn to ^4.0.0 and spacejam is still launching meteor !

Meteor pid and MongoDB pid are not correctly retrieved - cross-spawn

On windows and when you use cross-spawn, it is cmd.exe that launch node. So when you retrieve the meteor pid you get the cmd.exe pid and you get the node pid instead of the mongodb pid. What a mess... Did you saw such issue before ?
When I run spacejam on windows I need to kill zombie processes to rerun spacejam again.

[jsep]

@serut thanks for insigth I didn't saw that problem, working on it.

[serut]

@jsep I've implemented the fix to retrieve the good meteor pid : https://github.com/serut/spacejam/commit/c3c60c2cdea47fae07ecde2e63846320ff28f5ea
I also found out that if I put coffee-script as a dependency of spacejam we do not have to execute npm install -g [email protected] anymore ! ;-)
And I do not have the Native thread-sleep not available. issue, don't know why.

Tested on windows 10 (no zombie process), OS X and travis.

[serut]

I have again that warning on my log :

Native thread-sleep not available.
This will result in much slower performance, but it will still work.
You should re-install spawn-sync or upgrade to the lastest version of node if possible.

So I added "try-thread-sleep": "1.0.0" (otherwise need to update node to any version > 0.10) (repro here on the first test : codecov)

Also, on windows, I have a strange behaviour when coffeescript files are compiled. There are two processes to dig to get the mongo pid, but if I compile (with cake) lib/*.coffee into js files there are 4 processes to dig to get the mongo pid (I commited that yesterday). So I remove that compilation and I update the algorithm to dig only on 2 processes. (is it understandable ? ;-) ). I hope this time it will be stable. It looks good on travis, circleci and windows, but some feedback would be helpful !

Here is the diff : windows-suppport...serut:windows-suppport-pr

[jsep]

Hello @serut thanks, I was testing your code and I notice that too. Now I'm going to test your new changes.

Thanks a lot!

[serut]

it's definitely a perfect release for me, it does its job so far. Tested on Windows, Travis, Circle, OSX, Meteor 1.4.0.X

[serut]

@jsep @rbabayoff any improvement with this PR ? Do you need some help with these spacejam tests ?
My pull request with coverage is also quite good if you want to look at