-
-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
steam: add extraCompatPackages #293564
steam: add extraCompatPackages #293564
Conversation
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.
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.
Couple minor things but otherwise the diff LGTM.
I'll hopefully have time to test it this week but it really ought to work.
nixos/modules/programs/steam.nix
Outdated
luxtorpeda | ||
proton-ge |
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.
Are these two in Nixpkgs? If not, remove them or add a note about that and where to get these.
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.
as written in the pr description, there is no proton derivation in nixpkgs atm, so i removed that.
92d878b
to
d24c0d5
Compare
d24c0d5
to
93a891f
Compare
nixos/steam: fix eval after #293564
Successfully created backport PR for |
@Lord-Valen be aware that there is some refactoring ongoing for improving how compat tools could be defined, not sure if that should go to stable alone. see #296009 |
(cherry picked from commit 7b8d88f)
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin release-23.11
git worktree add -d .worktree/backport-293564-to-release-23.11 origin/release-23.11
cd .worktree/backport-293564-to-release-23.11
git switch --create backport-293564-to-release-23.11
git cherry-pick -x 93a891f0e8e0feb96090eac48d9a2d5aaad28f20 |
(cherry picked from commit 7b8d88f)
Description of changes
This PR adds an extra option for the steam module to define compat tools via declaration.
One has to provide a valid proton derivation to let it appear in the steams proton list.
What defines a valid proton derivation could be looked up here: https://gitlab.steamos.cloud/steamrt/steam-runtime-tools/-/blob/393bb134fc01f96d68925d6bfe300d58b97744ac/docs/steam-compat-tool-interface.md
This supersedes #189398. The option declaration is the same, but does not apply the environment variable globally but just on the steam wrapper.
For testing purposes one could either use the proton-ge that is have in my personal flake https://github.com/Shawn8901/nix-configuration/blob/main/packages/proton-ge-custom/default.nix or that one which is in the NUR https://github.com/nix-community/nur-combined/blob/master/repos/ataraxiasjel/pkgs/proton-ge/default.nix
sadly there is at the moment no custom proton runtime in nixpkgs at the moment (or at least i did not find any).
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 馃憤 reaction to pull requests you find important.