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
duckstation: init at unstable-2020-12-28 #107878
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.
Left some suggestions
Please also squash all the commits into one once your are done and rewrite the final commit message to read |
92e48d4
to
925eebe
Compare
@prusnak Done. Thank you. |
You might consider opening an issue upstream to replace |
It seems the UI is written in Qt, they use GTK2 only for the native file-selector dialog. 🤷 We might be able to patch that out and use Qt file-selector dialog instead, but I am unsure this will be accepted by upstream. |
It may be trivial to split the outputs for the sdl and qt binaries, this way |
Actually, I've just experimented with a gtk3 patch. I'm confirming that it works and update this PR + upstream PR. |
Rational: gtk2 is not maintained anymore and the change was simple. - Update of nativefiledialog/CMakeLists.txt in order to detect and use gtk3 - Updated a few files referencing gtk2, especially: - github workflow: I tested that the apt-get update works on ubuntu 18.03 in a docker container - Readme. This was tested in the following nixpkgs PR: NixOS/nixpkgs#107878
Upstream PR: stenzek/duckstation#1344 |
@prusnak I addressed your last batch of comment. Thank you for the time you take to improve the quality of my PR. |
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.
LGTM 👍
Result of 1 package built:
|
Motivation for this change
duckstation
is a nice playstation one emulator with more features than pcsxr or epsxe available in nixpkgs.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)