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
Wine update: 5.11 -> 5.12 #92295
Wine update: 5.11 -> 5.12 #92295
Conversation
Motivation: it works faster, and don't issue suspicious errors like `sed: couldn't flush stdout: Broken pipe` during applying patches.
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.
From the user point of view it looks good to me: built it and used it to run the Windows applications I normally use with Wine and they work (as well as before). One of them is foobar2000 so this also tests the faudio update to some extent.
From a style point of view I'm not the best person to ask, but I have one suggestion.
pkgs/misc/emulators/wine/staging.nix
Outdated
@@ -1,4 +1,4 @@ | |||
{ stdenv, callPackage, wineUnstable }: | |||
{ lib, stdenv, callPackage, wineUnstable }: |
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.
The other lib
in this file are used as stdenv.lib
, so do we want it as an extra argument for concatMapStringsSep
?
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.
ok, will use stdenv.lib
as well
Due repeatable issues with xactengine-initial patchset. Later, `disablePatchsets` can be extended to be overridable argument of wineStaging.
Motivation for this change
Keep wine unstable/staging up to date.
Also allows to disable specific patchsets from staging
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)