Skip to content

Conversation

@Hythera
Copy link
Member

@Hythera Hythera commented Nov 30, 2025

Diff: modrinth/code@v0.10.5...v0.10.23

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.

@nixpkgs-ci nixpkgs-ci bot requested a review from getchoo November 30, 2025 14:02
@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 30, 2025
@Hythera

This comment was marked as outdated.

@Hythera

This comment was marked as outdated.

@Eveeifyeve
Copy link
Member

Testing on real hardware and vm for linux without github action.

Copy link
Member

@Eveeifyeve Eveeifyeve left a comment

Choose a reason for hiding this comment

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

Please fix the issues.

@Eveeifyeve

This comment was marked as outdated.

@Hythera Hythera force-pushed the pkgs/modrinth-app-unwrapped/0.10.21 branch from 36b09bc to 07ee2e2 Compare December 1, 2025 06:19
@encode42

This comment was marked as outdated.

@Hythera
Copy link
Member Author

Hythera commented Dec 1, 2025

Still doesn't build on darwin with the same error.

@Eveeifyeve

This comment was marked as outdated.

@Hythera

This comment was marked as outdated.

@Eveeifyeve
Copy link
Member

Eveeifyeve commented Dec 1, 2025

I seems it cidre causing issues to build not finding a dir.

error: failed to run custom build command for `cidre v0.11.3`

Caused by:

process didn't exit successfully: /nix/build/nix-14596-1689605405/source/target/release/build/cidre-5bfc87287549a064/build-script-build (exit status: 101)

--- stdout

cargo:rustc-link-lib=clang_rt.osx

cargo:rustc-link-search=/nix/store/qk20nysrc2170f1mal5k6r0axqn6jmfj-clang-wrapper-21.1.2/resource-root/lib/darwin

cargo:rerun-if-changed=./pomace/

--- stderr

thread 'main' (164539) panicked at /nix/build/nix-14596-1689605405/modrinth-app-unwrapped-0.10.21-vendor/cidre-0.11.3/build.rs:131:14:

called Result::unwrap() on an Err value: Os { code: 2, kind: NotFound, message: "No such file or directory" }

note: run with RUST_BACKTRACE=1 environment variable to display a backtrace

warning: build failed, waiting for other jobs to finish...

Error failed to build app: failed to build app

@Hythera
Copy link
Member Author

Hythera commented Dec 1, 2025

Yea I looked at the project a bit and the line that was causing it. Not really sure what the root issue is tho. As I don't have a darwin device / experience with it...

@Eveeifyeve
Copy link
Member

Eveeifyeve commented Dec 1, 2025

It needs xcodebuild looking at the build.rs: https://github.com/yury/cidre/blob/6239660a051393c2f6d18ea969a7183e13ce6258/cidre/build.rs#L105-L132

I would say add xcbuild to this package and it should fix it.

@SuperSandro2000
Copy link
Member

Please rebase after #463705 got merged

@Hythera
Copy link
Member Author

Hythera commented Dec 1, 2025

#463705 did not fix the darwin build tho?

@Eveeifyeve
Copy link
Member

Eveeifyeve commented Dec 1, 2025

#463705 did not fix the darwin build tho?

So please rebase to pull all the changes from the pr, wtth updating it and then add xcbuild to your buildInputs.

@nixpkgs-ci nixpkgs-ci bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Dec 1, 2025
@Hythera Hythera force-pushed the pkgs/modrinth-app-unwrapped/0.10.21 branch 3 times, most recently from 35f1d47 to c30e9be Compare December 1, 2025 19:11
@nixpkgs-ci nixpkgs-ci bot removed the 2.status: merge conflict This PR has merge conflicts with the target branch label Dec 1, 2025
@Eveeifyeve
Copy link
Member

Eveeifyeve commented Dec 1, 2025

There is still issues with the dep.json please see the failing checks for more info.

@Hythera
Copy link
Member Author

Hythera commented Dec 1, 2025

I know, I'm updating it right now :)

@Hythera Hythera force-pushed the pkgs/modrinth-app-unwrapped/0.10.21 branch from c30e9be to dc62f40 Compare December 1, 2025 19:22
@encode42
Copy link
Member

encode42 commented Dec 1, 2025

Should the commit/PR name be changed since it's based on 0.10.5 now?

@encode42

This comment was marked as outdated.

@Hythera Hythera force-pushed the pkgs/modrinth-app-unwrapped/0.10.21 branch from dc62f40 to f3cab1d Compare December 1, 2025 20:04
@Hythera Hythera changed the title modrinth-app-unwrapped: fix build, 0.10.5 -> 0.10.21 modrinth-app-unwrapped: fix build, 0.10.5 -> 0.10.22 Dec 21, 2025
@Hythera
Copy link
Member Author

Hythera commented Dec 21, 2025

nixpkgs-review result for #466573

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 466573
Commit: 9990e0d6a3eae412c4a220c2e154bdcd20df4ea9 (subsequent changes)
Merge: 12f02faefe29514f8626dc6e2892aa7c33a7844c

Logs: https://github.com/Hythera/nixpkgs-review-gha/actions/runs/20413929627


x86_64-linux

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

@encode42
Copy link
Member

p.s. you might want to update this to 0.10.23 now, since it fixes a critical bug with NeoForge >1.21.10 installations!

@Hythera Hythera force-pushed the pkgs/modrinth-app-unwrapped/0.10.21 branch from 9990e0d to 9886c55 Compare December 21, 2025 20:34
@Hythera
Copy link
Member Author

Hythera commented Dec 21, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 466573
Commit: 9886c55c71da9f00295d25af884be66afef1cc14


x86_64-linux

✅ 3 packages built:
  • modrinth-app
  • modrinth-app-unwrapped
  • nixpkgs-manual

@Hythera Hythera changed the title modrinth-app-unwrapped: fix build, 0.10.5 -> 0.10.22 modrinth-app-unwrapped: fix build, 0.10.5 -> 0.10.23 Dec 21, 2025
@Hythera Hythera force-pushed the pkgs/modrinth-app-unwrapped/0.10.21 branch 2 times, most recently from 67d2234 to 5c5e542 Compare December 26, 2025 18:04
@Hythera Hythera requested a review from Eveeifyeve December 26, 2025 18:04
@Hythera Hythera force-pushed the pkgs/modrinth-app-unwrapped/0.10.21 branch from 5c5e542 to 990c4a3 Compare December 26, 2025 18:09
@Hythera Hythera changed the title modrinth-app-unwrapped: fix build, 0.10.5 -> 0.10.23 modrinth-app-unwrapped: 0.10.5 -> 0.10.23 Dec 26, 2025
@Hythera Hythera added the 8.has: package (update) This PR updates a package to a newer version label Dec 26, 2025
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. and removed 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. labels Dec 26, 2025
@Hythera
Copy link
Member Author

Hythera commented Dec 26, 2025

I've marked darwin as broken for now as it is blocking the update for the Linux users. Sorry for those who had to wait...

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 466573
Commit: 990c4a36dc9fd788347cedacbcd0996e017b765b (subsequent changes)
Merge: a49d63c6961519a4dc52b36b4989b913fb820e84

Logs: https://github.com/Hythera/nixpkgs-review-gha/actions/runs/20527083292


x86_64-linux

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

aarch64-linux

No rebuilds


x86_64-darwin

No rebuilds


aarch64-darwin

No rebuilds

@Scrumplex Scrumplex added this pull request to the merge queue Dec 26, 2025
@Scrumplex Scrumplex added the backport release-25.11 Backport PR automatically label Dec 26, 2025
@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 2 This PR was reviewed and approved by two persons. 2.status: merge-bot eligible This PR can be merged by commenting "@NixOS/nixpkgs-merge-bot merge". and removed 12.approvals: 1 This PR was reviewed and approved by one person. labels Dec 26, 2025
Merged via the queue into NixOS:master with commit e1e99ec Dec 26, 2025
33 of 37 checks passed
@nixpkgs-ci
Copy link
Contributor

nixpkgs-ci bot commented Dec 26, 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 26, 2025
@Hythera Hythera deleted the pkgs/modrinth-app-unwrapped/0.10.21 branch December 28, 2025 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.status: merge-bot eligible This PR can be merged by commenting "@NixOS/nixpkgs-merge-bot merge". 8.has: package (update) This PR updates a package to a newer version 8.has: port to stable This PR already has a backport to the stable release. 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. 12.approvals: 2 This PR was reviewed and approved by two persons. backport release-25.11 Backport PR automatically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants