Skip to content
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

pcsx2: 1.7.5004 -> 1.7.5318 #273330

Merged
merged 1 commit into from
Dec 22, 2023
Merged

pcsx2: 1.7.5004 -> 1.7.5318 #273330

merged 1 commit into from
Dec 22, 2023

Conversation

SuperSamus
Copy link
Contributor

@SuperSamus SuperSamus commented Dec 10, 2023

Description of changes

Update to the latest version.

One thing I'm not able to implement: one of the changes made upstream is the disabling of Wayland, but you can enable it using the I_WANT_A_BROKEN_WAYLAND_UI environment variable.
This is the best I could do (inside qtWrapperArgs):

    ''--set QT_QPA_PLATFORM "\''$(if [[ -z $I_WANT_A_BROKEN_WAYLAND_UI ]]; then echo 'xcb'; else echo \'\'; fi)"''

Can someone more knowledgeable of Bash and escape hells fix this?

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@qubitnano
Copy link
Contributor

Result of nixpkgs-review pr 273330 run on x86_64-linux 1

1 package built:
  • pcsx2

@qubitnano
Copy link
Contributor

That wayland env seems to just be for the upstream appimage. Wayland is still enabled by default in the cmake flags between these two version.

option(WAYLAND_API "Enable Wayland support" ON)

And from nixpkgs-review build log

-- Found Wayland_Client: /nix/store/pdgavpdmzfwf90k46ybca08bwlc3hzdc-wayland-1.22.0/lib/libwayland-client.so (found version "1.22.0") 
-- Found Wayland_Egl: /nix/store/pdgavpdmzfwf90k46ybca08bwlc3hzdc-wayland-1.22.0/lib/libwayland-egl.so (found version "18.1.0") 
-- Found Wayland: /nix/store/pdgavpdmzfwf90k46ybca08bwlc3hzdc-wayland-1.22.0/lib/libwayland-client.so;/nix/store/pdgavpdmzfwf90k46ybca08bwlc3hzdc-wayland-1.22.0/lib/libwayland-egl.so (found version "1.22.0") found components: Egl

On KDE I get the wayland app icon when running pcsx2-qt

@SuperSamus SuperSamus changed the title pcsx2: 1.7.5004 -> 1.7.5270 pcsx2: 1.7.5004 -> 1.7.5288 Dec 13, 2023
@SuperSamus
Copy link
Contributor Author

Alright, I managed to implement I_WANT_A_BROKEN_WAYLAND_UI correctly.

@SuperSamus SuperSamus changed the title pcsx2: 1.7.5004 -> 1.7.5288 pcsx2: 1.7.5004 -> 1.7.5290 Dec 14, 2023
@SuperSamus SuperSamus changed the title pcsx2: 1.7.5004 -> 1.7.5290 pcsx2: 1.7.5004 -> 1.7.5298 Dec 15, 2023
@SuperSamus SuperSamus changed the title pcsx2: 1.7.5004 -> 1.7.5298 pcsx2: 1.7.5004 -> 1.7.5312 Dec 19, 2023
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/3124

@hrdinka
Copy link
Contributor

hrdinka commented Dec 21, 2023

Hi, thank you very much for your (ongoing) contributions!

If I understand it correctly, I_WANT_A_BROKEN_WAYLAND_UI is redundant to QT_QPA_PLATFORM=xcb but added to be compatible with the AppImage/official documentation, is this right?

@SuperSamus SuperSamus changed the title pcsx2: 1.7.5004 -> 1.7.5312 pcsx2: 1.7.5004 -> 1.7.5318 Dec 21, 2023
@SuperSamus
Copy link
Contributor Author

Yes.

@hrdinka hrdinka merged commit 17bb289 into NixOS:master Dec 22, 2023
@hrdinka
Copy link
Contributor

hrdinka commented Dec 22, 2023

Tested it locally, could however not test with wayland.

Thank you very much for your great work! 👍

@SuperSamus SuperSamus deleted the pcsx2-update branch December 22, 2023 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants