-
-
Notifications
You must be signed in to change notification settings - Fork 63
[bugfix] Fix mod installation on NixOS #870
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
[bugfix] Fix mod installation on NixOS #870
Conversation
|
@ImSapphire I might double check on the flatpak build if I can install mods with this. There are some weird env vars that get injected there that I assume might break flatpak. @Mistyttm can you help me verify this if everything's working as expected since your on NixOS. |
@silentrald can already confirm it is my friend, these patches were on the nixos package before Sapphire brought them here. They managed to get it all working |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be good to merge (ノ^ヮ^)ノ*
@ImSapphire would this needs to be merged first or the other PR from NixOS? Or they are independent to one another.
@silentrald They are independent as of now. We're pulling from the tagged 1.5.3 release, so when the tagged 1.5.4 release with this code comes out, we can update the package on nixpkgs and remove the patches on our end nixpkgs version is completely independent from this :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 🙌
Merging! 🚀 |
Scope
Inherits environment when starting wine for BSIPA installation, which matches behaviour when launching Beat Saber.
Use selected Proton's wine instead of system wine, since the wine prefix version may mismatch and cause wine to crash.
Emoji Guide
For reviewers: Emojis can be added to comments to call out blocking versus non-blocking feedback.
E.g: Praise, minor suggestions, or clarifying questions that don’t block merging the PR.
E.g: Blocking feedback must be addressed before merging.