Skip to content
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

Windows compatible? Coffeescript? #8

Open
fabyeah opened this issue Oct 12, 2015 · 5 comments
Open

Windows compatible? Coffeescript? #8

fabyeah opened this issue Oct 12, 2015 · 5 comments

Comments

@fabyeah
Copy link

fabyeah commented Oct 12, 2015

When I try to run packagecheck it says The system cannot find the path specified.

Then I changed all the bash stuff in packagecheck, packagecheck.cmd and node_modules/packagecheck/run to node and now it at least runs the run file, but says SyntaxError for the # coffeescript comments.

I removed all of those and then I get

ulimit -Sn 2048
       ^^^^
SyntaxError: Unexpected number

What do I do?
(Im on Win10 btw)

@timothyarmes
Copy link
Owner

I've never tested on Windows I'm afraid, and I don't have a Windows box the develop on.

The ulimit call may not be necessary on Windows at all. You could try to conditionally remove it.

If you get it working then PRs are accepted :)

@fabyeah
Copy link
Author

fabyeah commented Oct 12, 2015

When the ulimit call is removed there are more syntax errors in run still.

I tried running index.js directly in packagecheck and packagecheck.cmd, which will run the script, but output [TypeError: Cannot read property 'semver' of undefined] for every dependency of every package...? :(

Is there any workaround or alternative to this package?

@akanix42
Copy link

akanix42 commented Feb 4, 2016

It's working for me on Windows, with no errors.
To clarify @fabyeah's comment, to get it working on Windows just replace the contents of packagecheck.cmd with the following snippet:

@IF EXIST "%~dp0\node.exe" (
  "%~dp0\node.exe"  "%~dp0\node_modules\packagecheck\index.js" %*
) ELSE (
  @SETLOCAL
  @SET PATHEXT=%PATHEXT:;.JS;=;%
  node  "%~dp0\node_modules\packagecheck\index.js" %*
)

@timothyarmes
Copy link
Owner

I would be good if this could be made to automatically do the right thing based on Mac/Windows.

@fabyeah
Copy link
Author

fabyeah commented Feb 19, 2016

@nathantreid thanks a ton! It seems to work now, though I'm getting a new issue #11, hopefully unrelated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants