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
protontricks: 1.4.4 -> 1.5.0 #120612
protontricks: 1.4.4 -> 1.5.0 #120612
Conversation
Result of 1 package built:
|
c8f6d94
to
f5ff8b2
Compare
- Update steam-run.patch to support Pressure Vessel runtime - Fix adding $PROTON_DIST_PATH/lib to LD_LIBRARY_PATH when using legacy runtime - Use bash instead of /bin/sh now that wrappers are non-POSIX compliant - Remove `test_run_steam_runtime_not_found` in steam-run.patch instead of using disabledTests - Add import check
f5ff8b2
to
27f60ec
Compare
Sorry for the force-pushes. I noticed that the non-bwrap environment was using the scout runtime from |
Result of 1 package built:
|
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.
It would be great for reviewing if the patch was a bit smaller.
@SuperSandro2000 Hmm, the only changes that are unnecessary are the unused legacy runtime functions that I removed (their purpose is replaced by the fhs environment setup by The diff probably looks so big because upstream changed so much in 1.5.0. Maybe it would be easier to just look through just the patch file instead of the diff between the old and new patch file? |
One thing I just wanted to mention is that with the nixpkgs/pkgs/tools/package-management/protontricks/steam-run.patch Lines 166 to 181 in 27f60ec
And updated the legacy runtime code path to use nixpkgs/pkgs/tools/package-management/protontricks/steam-run.patch Lines 245 to 250 in 27f60ec
|
$PROTON_DIST_PATH/lib
toLD_LIBRARY_PATH
when using legacy runtimebash
instead of/bin/sh
now that wrappers are non-POSIX complianttest_run_steam_runtime_not_found
in steam-run.patch instead of usingdisabledTests
Motivation for this change
Update to the latest version: https://github.com/Matoking/protontricks/releases/tag/1.5.0 (fixes #119150)
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)