Skip to content

Comments

Heroic Games Launcher: 2.14.1 -> 2.15.1#335737

Merged
K900 merged 2 commits intoNixOS:masterfrom
aidalgol:heroic-2.15
Aug 21, 2024
Merged

Heroic Games Launcher: 2.14.1 -> 2.15.1#335737
K900 merged 2 commits intoNixOS:masterfrom
aidalgol:heroic-2.15

Conversation

@aidalgol
Copy link
Contributor

@aidalgol aidalgol commented Aug 19, 2024

Description of changes

Update Heroic Games Launcher to latest release.

Rolled #327273 into this PR.

Narrows meta.platforms to only x86_64 Linux. For Linux, only x86_64 is supported by upstream, and for nix-darwin, this really needs a dedicated maintainer. It will also probably not work well compared to using upstream's macOS build. This package is really only meant for NixOS. If you are on another operating system, using this nix package will be more trouble than using an official build, or an unofficial package specifically for your platform.

Closes #333956.

Supersedes #327273

Fixes #327153

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.11 Release Notes (or backporting 23.11 and 24.05 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.

@aidalgol aidalgol requested a review from K900 August 19, 2024 05:43
@aidalgol aidalgol mentioned this pull request Aug 19, 2024
13 tasks
@ofborg ofborg bot added 8.has: clean-up This PR removes packages or removes other cruft 8.has: package (new) This PR adds a new package labels Aug 19, 2024
@ofborg ofborg bot requested review from teutat3s and yayayayaka August 19, 2024 06:15
@ofborg ofborg bot added 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. labels Aug 19, 2024
@teutat3s
Copy link
Member

PR #333907 has been merged, this can be rebased.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did this regress with this update? If so fine for me. If it theoretically could still work and you just didn't test it, please keep the platform here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was a change to the upstream build that puts x64 in the path to the helper binaries, so it looks as if they are starting to hard-code assumptions about the platform. As for ARM Linux and Darwin in general, Heroic may theoretically run, but any game you try to launch almost certainly will not, and that is the entire point of the application.

@aidalgol aidalgol marked this pull request as ready for review August 20, 2024 09:39
@yayayayaka
Copy link
Member

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

2 packages built:
  • heroic
  • heroic-unwrapped

Copy link
Member

@yayayayaka yayayayaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nixpkgs-review happy and application is launching. Looking fine by me.

@ofborg ofborg bot added 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. and removed 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. labels Aug 20, 2024
@wegank wegank added the 12.approvals: 1 This PR was reviewed and approved by one person. label Aug 20, 2024
@nadir-ishiguro
Copy link
Contributor

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

2 packages built:
  • heroic
  • heroic-unwrapped

@nadir-ishiguro
Copy link
Contributor

Runs as expected and games seem to run as well (only tested a few).

@K900 K900 merged commit ab13423 into NixOS:master Aug 21, 2024
@github-actions
Copy link
Contributor

Successfully created backport PR for release-24.05:

@aidalgol aidalgol deleted the heroic-2.15 branch January 8, 2025 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: clean-up This PR removes packages or removes other cruft 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update request: Heroic 2.14.1 → 2.15.1 Stardew Valley doesn't open in Heroic Launcher

8 participants