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
can't install purescript #3544
Comments
This should be reported to https://github.com/purescript-contrib/node-purescript/ |
@justinwoo No. |
The binary I download from the release works via three ways:
|
From https://github.com/purescript-contrib/node-purescript/issues/65:
@shinnn I've just checked on my own linux system and the v0.12.3 binary is working normally, so I don't think 1) is correct. I wonder if the download might have gone wrong somehow, or are you perhaps on a different architecture (i.e. 32-bit vs 64-bit)? @sagax can you try downloading the binary manually from that url and executing it? Also, can you paste the output of The |
Please, check the log posted by the issue author before participating in the discussion.
|
@shinnn I did check the log. Notice that the error is "cannot execute binary file", which is why I suggested checking the architecture. See https://superuser.com/questions/435988/how-can-i-resolve-the-error-cannot-execute-binary-file. Does the node purescript package check that system architecture matches the binary? |
@shinnn also, it's not really fair of you to assume that @justinwoo didn't check the log either. Either way the fact that it works on both of our systems is useful context. Clearly the binary is not totally broken, and so more investigation is needed to work out what's actually gone wrong. |
If INSTALL.md clearly states the required system architecture, of course it will. If there is no mention about it in the documentation, users and integrated tools generally don't assume the binary works only on 64-bit systems. |
The compiler should work on all the architectures that GHC supports. However, any given compiled binary will usually only work on one architecture. I think it's pretty rare for a single binary to run on more than one architecture in general. In the case of our binaries, there are some pretty big clues in the file names: "linux64", "win64". Should I take that as a "no", the node purescript package doesn't check that the system architecture matches the architecture the binary is expecting? If so, I think it should. |
Or at least, 32-bit binaries can often run on 64-bit systems, but not vice versa. |
If PureScript is a closed-source personal hobby, it' ok. But PureScript is a language, where this kind of implicitness is not allowed, I think. Anyway, thanks for taking your time to approve this is not actually a bug of the npm package but more like room for UX improvement of it and a documentation issue of this repository. |
I’m going to close this as it seems like this is not a compiler issue. |
Though this is not a compiler issue in a sense, again, it's a documentation issue of this repository. |
Oh yes of course, thanks. I’ll update INSTALL.md later today. |
- Clarify architecture targeted by prebuilt binaries, refs #3544 - Separate operating system requirements from prebuilt binaries section - Explain how to install an official binary bundle - Clarify difference between official binary bundles and other compiler distributions - Remove outdated reference to "other utilities" included in an official binary bundle (now it's just `purs`) - Use proper link text for stack install documentation link - Remove outdated sentence about `stack setup`, since stack now installs the appropriate GHC for you automatically by default.
- Clarify architecture targeted by prebuilt binaries, refs #3544 - Separate operating system requirements from prebuilt binaries section - Explain how to install an official binary bundle - Clarify difference between official binary bundles and other compiler distributions - Remove outdated reference to "other utilities" included in an official binary bundle (now it's just `purs`) - Use proper link text for stack install documentation link - Remove outdated sentence about `stack setup`, since stack now installs the appropriate GHC for you automatically by default.
command
simple log https://termbin.com/yene
complete log https://termbin.com/kzs7
The text was updated successfully, but these errors were encountered: