Skip to content

Conversation

@gepbird
Copy link
Contributor

@gepbird gepbird commented Nov 21, 2025

ZHF: #457852

Relases:

This is not the latest version, just the earliest one to fix the build.

cc @typedrat (#462979 (comment))

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@gepbird gepbird requested a review from getchoo November 21, 2025 10:24
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. labels Nov 21, 2025
@gepbird gepbird added the 0.kind: ZHF Fixes Fixes during the Zero Hydra Failures (ZHF) campaign label Nov 21, 2025
@gepbird
Copy link
Contributor Author

gepbird commented Nov 21, 2025

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 463705
Commit: 1cfc956f8d3fd857efa2b3d0a11e3642aca0bff6 (subsequent changes)
Merge: b0bd8fe6c7b681ade07a3c99b793c91588f62b80

Logs: https://github.com/gepbird/nixpkgs-review-gha/actions/runs/19567433212

Download packages from cache:
  • x86_64-linux
    nix-store -r --add-root nixpkgs-pr-463705-x86_64-linux \
      --option binary-caches 'https://cache.nixos.org/ https://gepbird-nixpkgs-review-gha.cachix.org' \
      --option trusted-public-keys '
      cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
      gepbird-nixpkgs-review-gha.cachix.org-1:DXx7aJqJyRfrgSacf/SSVzEaPbrH3lych7ti/zUBol4=
      ' \
      /nix/store/bikl8q2y5qzxw2vv1wms2lb1v6cch81q-modrinth-app-unwrapped-0.10.5 \
      /nix/store/iymkvq6pqwzy4ygqlx9cdq4n2s6r7fyr-modrinth-app-0.10.5

x86_64-linux

✅ 2 packages built:
  • modrinth-app
  • modrinth-app-unwrapped

aarch64-linux

No rebuilds


x86_64-darwin (sandbox = relaxed)

❌ 2 packages failed to build:
  • modrinth-app
  • modrinth-app-unwrapped

aarch64-darwin (sandbox = relaxed)

❌ 2 packages failed to build:
  • modrinth-app
  • modrinth-app-unwrapped

@gepbird

This comment was marked as resolved.

@gepbird gepbird marked this pull request as draft November 21, 2025 11:52
@Hythera
Copy link
Member

Hythera commented Nov 21, 2025

Tested on x86_64-linux, most features like log-in, creating and starting a modpack work. Downloading mods and other content doesn't (because of the mentioned missing env vars). Java fails to download and isn't picked up by default. :)

@gepbird gepbird marked this pull request as ready for review November 21, 2025 15:54
@gepbird
Copy link
Contributor Author

gepbird commented Nov 21, 2025

Tested on x86_64-linux, most features like log-in, creating and starting a modpack work. Downloading mods and other content doesn't (because of the mentioned missing env vars). Java fails to download and isn't picked up by default. :)

Thanks, that's very helpful! That sounds better than it being broken.

In the future I plan to maintain this (and use it regularly, I'm still on Prism) :)

@gepbird gepbird force-pushed the fix/modrinth-app-unwrapped/rust-1.90 branch from 1cfc956 to db2f3c5 Compare November 21, 2025 17:42
@Hythera
Copy link
Member

Hythera commented Nov 21, 2025

(non-blocking) Everything seems to work for me now. I only ran into two issues. Firstly jumping into worlds doesn't work from the home page (only from the modpack page), this will probably be fixed by updating the launcher to a newer version. Secondly the launcher is not able to download its own java installation (there are some issues relating that on modrinth's repository).

JRE error: Could not check Java version at path /home/username
/.local/share/ModrinthApp/meta/java_versions/zulu21.46.19-ca-jre21.0.9-linux_x64/bin/java

Setting the java path manually fixes this.

Edit: I tried updating the package locally to 0.10.20 and it works without any extra steps. This also fixes issue 1.

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Nov 22, 2025
@gepbird gepbird added the backport release-25.11 Backport PR automatically label Nov 24, 2025
Copy link
Member

@mdaniels5757 mdaniels5757 left a comment

Choose a reason for hiding this comment

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

FYI: v0.10.21 is now out (https://github.com/modrinth/code/releases).

I see that you mentioned that you had only updated as little as possible in order to fix the build. May I ask why? Normally I try to update as much as possible in this situation (where the stable branch has never had a working build, so breaking changes are not an issue).

@gepbird
Copy link
Contributor Author

gepbird commented Nov 29, 2025

Feel free to update it to the latest. I did not hane the capacity to read through all those changes, possibly encounter and fix other issues. Plus a smaller change could be easier and faster to review.

@mdaniels5757
Copy link
Member

Fair enough, I just thought that I should mention it, since Hythera above reported that 0.10.20 fixed some issues they had (and they didn't say it needed any changes other then version/hashes)

@SuperSandro2000 SuperSandro2000 added this pull request to the merge queue Dec 1, 2025
Merged via the queue into NixOS:master with commit ff0f587 Dec 1, 2025
33 checks passed
@nixpkgs-ci
Copy link
Contributor

nixpkgs-ci bot commented Dec 1, 2025

Successfully created backport PR for release-25.11:

@github-actions github-actions bot added the 8.has: port to stable This PR already has a backport to the stable release. label Dec 1, 2025
@gepbird gepbird deleted the fix/modrinth-app-unwrapped/rust-1.90 branch December 1, 2025 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0.kind: ZHF Fixes Fixes during the Zero Hydra Failures (ZHF) campaign 8.has: port to stable This PR already has a backport to the stable release. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person. backport release-25.11 Backport PR automatically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants