Skip to content

[Backport release 24.11] gradle 8.10.2 -> 8.12#381494

Closed
britter wants to merge 9 commits intoNixOS:release-24.11from
britter:backport-356109-to-release-24.11
Closed

[Backport release 24.11] gradle 8.10.2 -> 8.12#381494
britter wants to merge 9 commits intoNixOS:release-24.11from
britter:backport-356109-to-release-24.11

Conversation

@britter
Copy link
Contributor

@britter britter commented Feb 12, 2025

Manual backport of #356109, after automatic backport failed.

Cherry-picking resulted in a conflict in keyguard/deps.json. I accepted the version from master and then ran the dependency update again based on my backport branch.

@github-actions github-actions 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 Feb 12, 2025
Copy link
Member

@FliegendeWurst FliegendeWurst 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 passes on x64 Linux.

LGTM assuming there were no breaking changes in Gradle 8.12. Besides the required deps.json updates.

@britter
Copy link
Contributor Author

britter commented Feb 13, 2025

The release notes for 8.12 mention a few potential breaking changes: https://docs.gradle.org/8.12/userguide/upgrading_version_8.html#potential_breaking_changes_2
Is that a problem?

@chayleaf
Copy link
Contributor

chayleaf commented Feb 13, 2025

Is that a problem?

Depends on whether build failures of downstream packages are within scope of nixpkgs backwards compatibility. If yes, then that is very much a problem (certain packages even break in-tree because of these updates).

@britter
Copy link
Contributor Author

britter commented Feb 13, 2025

8.12 release notes don't go into detail how the Kotlin release could break a build. I've reviewed the Kotlin 2.0.0 release notes and funny enough they don't even list what's breaking.

The other change about daemonJvm will definitely break a build using the old API. So I guess that's a show stopper.

@chayleaf
Copy link
Contributor

chayleaf commented Feb 13, 2025

Kotlin releases can break builds because Gradle package versions may depend on the bundled Kotlin version, a changed Kotlin version may result in a different dependency version, which means the new version won't be cached (I'm thinking of adding warnings if the build-time version doesn't match the lock-time version). However, this is only a problem in no-internet scenarios like Nix sandbox.

@wegank wegank added 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. 12.approvals: 2 This PR was reviewed and approved by two persons. labels Feb 14, 2025
@britter
Copy link
Contributor Author

britter commented Feb 14, 2025

@chayleaf this all sounds like we should close this, although @FliegendeWurst reported that nixpkgs-review was successful.

@FliegendeWurst
Copy link
Member

Actually I just checked again, it seems (at least) ghidra fails to build. Not sure why it did not pick that up previously.

@britter
Copy link
Contributor Author

britter commented Feb 15, 2025

Okay, it sounds like we should stick with 8.10.2 on release-24.11, although people will miss quite some Gradle releases until 25.05 is released.

@britter britter closed this Feb 15, 2025
@britter britter deleted the backport-356109-to-release-24.11 branch February 15, 2025 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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. 12.approvals: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants