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

Update Gradle Tooling API to 8.6 #6926

Merged

Conversation

lkishalmi
Copy link
Contributor

It's quite possible that Gradle 8.6 would be released before NB21, so let's start the upgrade now, and maybe add another PR for the final 8.6 during the RC cycle.

@lkishalmi lkishalmi added Gradle [ci] enable "build tools" tests Upgrade Library Library (Dependency) Upgrade labels Jan 7, 2024
@lkishalmi lkishalmi added this to the NB21 milestone Jan 7, 2024
Copy link
Member

@mbien mbien left a comment

Choose a reason for hiding this comment

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

looks good

@neilcsmith-net
Copy link
Member

It's quite possible that Gradle 8.6 would be released before NB21

And if it isn't, do we wait on it??

Making the IDE 8.6 aware is a good thing. But the tooling API is meant to be forwards compatible. What do we have that won't work with 8.6 even if we ship with 8.5 of the tooling API?

@lkishalmi
Copy link
Contributor Author

Well, I'm about to update the New Gradle project wizard and use: https://docs.gradle.org/8.6-rc-1/release-notes.html#generating-without-interactive-questions

Not strictly necessary, though...

@sdedic
Copy link
Member

sdedic commented Jan 11, 2024

Please wait with integration until the "Apache NBLS" release in the next few days, in case something screws up (as always). Thanks.

@mbien
Copy link
Member

mbien commented Jan 11, 2024

how can there be a release 5 days before NB feature freeze? There is no wiggle room to delay pending NetBeans PRs right now. They have to get in this weekend or they will miss NB 21.

@sdedic
Copy link
Member

sdedic commented Jan 11, 2024

how can there be a release 5 days before NB feature freeze? There is no wiggle room to delay pending NetBeans PRs right now. They have to get in this weekend or they will miss NB 21.

Sorry; I didn't match the different schedules right. I didn't meantto request a feature slip, the vsnetbeans is already branched anyway.

Copy link
Member

@sdedic sdedic left a comment

Choose a reason for hiding this comment

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

  • I have no opinion on (potentially) releasing with 8.6rc-1.
  • We need to plan testing in NBLS area during feature freeze. Our 'sample projects' are usually affected by gradle changes -> @MartinBalin

@neilcsmith-net
Copy link
Member

Well, I don't know why we're releasing a VSNetBeans release at the same time as 21-rc1, but that's a discussion for dev@ where it should have happened in the first place ...

On this change, @lkishalmi if there's a good reason to bring in an rc of Gradle tooling then go ahead IMO. As long as it doesn't "leak" into the UI I don't see a problem. ie. like the use of latest in init, the user shouldn't ever be aware we're using an rc or that potentially becomes a delay to our release.

@MartinBalin
Copy link
Contributor

VSNetBeans 20.0.301 is branched and being released. There is no problem with this PR making it to 21. If it breaks something issue will be filed. Svata just mentioned we have to test Gradle projects carefully during 21 testing as these are breaking more easily than Maven....

@neilcsmith-net
Copy link
Member

Well, is there a list of which areas of Gradle support are Tooling API version dependent? In theory the tooling API version shouldn't have an effect on projects. If we can communicate to testers during rc phase which features are dependent, we could potentially focus testing on upgrades?

@lkishalmi
Copy link
Contributor Author

Above: NetBeans 21-rc2 running a Java 21 Project with Gradle 8.6, where the Java Runtime for Gradle is Java 17.

Below: This PR running a Java 21 Project with Gradle 8.6, while the Java Runtime for Gradle is Java 21.

image

@lkishalmi lkishalmi changed the base branch from master to delivery February 3, 2024 17:12
@lkishalmi lkishalmi changed the title Update Gradle Tooling API to 8.6-rc-1 Update Gradle Tooling API to 8.6 Feb 3, 2024
@lkishalmi lkishalmi modified the milestones: NB22, NB21 Feb 3, 2024
@neilcsmith-net
Copy link
Member

neilcsmith-net commented Feb 4, 2024

NetBeans 21-rc2 running a Java 21 Project with Gradle 8.6, where the Java Runtime for Gradle is Java 17.

And how about running 21-rc2 with the IDE and Gradle runtime set to Java 21? I'm not against merging this change, even so late. However, isn't the main issue that we're warning people about JDK based on the Tooling API version rather than the project Gradle version?

Running on a clean userdir, a new Gradle project is currently set up with 8.5, and I presume will pick up 8.6 as and when the Gradle people designate it latest. There seems to be a problem that this version is not picked up on initial load from the wizard though? Is it loading before the wrapper is properly configured?

@MartinBalin
Copy link
Contributor

It has to be tested with latest micronaut-core which is the ultimate test for Gradle all the time.

@neilcsmith-net
Copy link
Member

@MartinBalin I intend to merge this after #7029 unless you have specific concerns now? Are there test micronaut-core projects to consider / look at? Note that this shouldn't change the version of Gradle used in those projects, unless they don't use the wrapper?

@MartinBalin
Copy link
Contributor

Yes, can be merged.

@neilcsmith-net neilcsmith-net merged commit 1ebbb7e into apache:delivery Feb 6, 2024
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Gradle [ci] enable "build tools" tests Upgrade Library Library (Dependency) Upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants