Darryl Miles
09/02/2024, 11:59 AMorg/gradle/internal/operations/BuildOperationExecutor#start(BuildOperationDescriptor.Builder descriptor) I note the comment above talks of using runner API but the #getRunner() was also removed around the same time.Vampire
09/02/2024, 1:52 PMDarryl Miles
09/02/2024, 1:56 PMDarryl Miles
09/02/2024, 1:58 PMThomas Broyer
09/02/2024, 2:07 PMDarryl Miles
09/02/2024, 2:08 PMVampire
09/02/2024, 3:41 PMTask.execute extension function, I guess it is a try to replace the long-gone Task.execute method that was there and was never intended to be public API.
Actually, searching with https://github.com/search?q=repo%3Adlmiles/gradle-xjc-plugin%20execute&type=code I do not even see that method be used anywhere.
The third link you shared seems to properly use the tooling api to run against different Gradle versions.
So maybe that extension function is just a left-over and not actually used anymore?Vampire
09/02/2024, 4:10 PMexecute() method was remove in 5.0.
But even in that commit from a cursory look I do not see a single spot using that function.
So it seems to me like it was a needless error right from the start to add that extension function, especially if it was really added without reason.
Also the IDE does not find any usages.Darryl Miles
09/02/2024, 4:33 PMVampire
09/02/2024, 4:54 PMexecute I don't think this is really directly related to Gradle removing the execute method. No idea why it was tried to replicate it. And even if it was used before, that was already a big mistake and replicating the method using internal API would only have been a quite bigger one. 🤷♂️ 🙂