-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Version 7.7.0
of keytar
breaks older Linux distributions (incl. RHEL7)
#9640
Comments
@vinokurig, do you have any thoughts about this? |
Hey, just chiming in, I'd have a few questions:
Is it a build or runtime issue? It's unclear.
Can you please share a few links with the community? What did you try?
Who do you consider as the user in the context of this issue:
Please share some details. Thank you! |
Runtime. I said build initially and then remembered that it was actually runtime, but was going to circle back and check. The build is successful, but startup fails.
atom/atom#21860 Most say either upgrade your distribution or install / compile a newer version of the standard library. We aren't able to do either, though many could. My team went with a
I've changed to the word 'adopter', but here are my thoughts:
The problem could affect someone in any of those positions (incl. the end-user, if the publisher didn't check / didn't publish a separate binary), since it's a runtime crash. |
Thanks! What about keeping the versions in-sync with VS Code: https://github.com/microsoft/vscode/blob/94463fabc296d242eaa80558630790012471cc8a/package.json#L67 |
@colin-grant-work My hope is that the build will either fail because of reasons which can be fixed and upstreamed, or the build will work and then the package should also work in the same environment it was built in. |
Bug Description:
The
keytar
package introduced to our dependencies in #9463 depends on different versions of the standard library depending on release. While7.6.0
works on RHEL7,7.7.0
doesn't, and kills the startup with an error like the following:Most advice online suggests that users upgrade their distribution, but that isn't always possible in a corporate context. Downstream, we can add this to our resolutions, but we might want to consider using
7.6.0
in the framework to keep things simple and reduce 'gotcha's for adopters working on various platforms unless there's a good reason to prefer7.7.0
.The text was updated successfully, but these errors were encountered: