-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Install fails on freebsd do to missing argon2 #4670
Comments
Woah, this looks new. Are you saying we should update the docs to tell users to install argon2 locally before installing code-server? |
node-rs now publishes an argon2 package. Might help here: https://github.com/napi-rs/node-rs/tree/main/packages/argon2 |
I'm running to this issue as well on 13.0-RELEASE-p6. Is there any additional information I can include to get a fix going? The install continues to fail on
|
Can you try using 4.2.0? |
It's the FreeBSD version. This open issue is for failed FreeBSD installations. I am using the latest code-server. |
I'm not super familiar with FreeBSD but we use I wonder if we could make @code-asher any ideas? |
Maybe we can go back to node-rs/argon2 now that we know how to work
around the platform issue.
|
Is that something I can platform test or help with in any way? |
Testing would be highly appreciated!
I think the steps to fix are:
1. Switch to node-rs/argon2. This is actually all that is needed to fix
the `npm install -g` case but there is a bit more we need to do to
fix our standalone releases hosted on GitHub.
2. Add `--ignore-platforms` to the `yarn install` commands in the
`package-linux-cross` and `package-macos-amd64` CI jobs so it
installs the binaries for every platform (the core issue is that
optional dependencies download for the *host* platform and not the
*target* platform so we have to just install all of them as a
workaround until that bug is sorted).
3. Optional: it would be nice to then remove the binaries for unrelated
platforms.
Alternatively we could try submitting a PR to node-argon2 to support
building for FreeBSD.
|
Try updating to |
Closing via #5492 |
OS/Web Information
code-server --version
:Steps to Reproduce
Expected
should install fine
Actual
Notes
This happen because argon didn't use to provide FreeBSD prebuild packages, however this is no longer the case, there binary node files now, and this includes FreeBSD:
https://github.com/napi-rs/node-rs/releases
The text was updated successfully, but these errors were encountered: