Integrated: 1713: Fix gradle deprecations
Erik Joelsson
erikj at openjdk.org
Mon Dec 5 21:44:01 UTC 2022
On Mon, 5 Dec 2022 19:34:07 GMT, Erik Joelsson <erikj at openjdk.org> wrote:
> This patch tries to fix the deprecation warnings currently printed by Gradle when building Skara.
>
> * enabled -> required: a simple rename of an attribute
> * TestReport was a little trickier, but as far as I can tell, this is how it's supposed to be done now, and it works just as well/bad as it did before this change. It's not clear to me what the testReport task is good for here, as it seems to me that it will only run if all tests are successful.
> * ModulePlugin.java: Was using some deprecated APIs. While at it I let Intellij fix some things that it was warning about. The null checks I removed were weird as the returned object was essentially an `Optional` (a different implementation of the same thing). Switched to using the modern instanceof assignment.
> * I updated gradle-wrapper.properties to match the version used by the Skara gradlew wrapper script. Intellij will by default pick the version from the properties file, so as long as we have that file around, we should make sure they match. I needed to do this to make the build succeed in Intellij as some of these changes didn't work in Gradle 7.2.
This pull request has now been integrated.
Changeset: 769c69b6
Author: Erik Joelsson <erikj at openjdk.org>
URL: https://git.openjdk.org/skara/commit/769c69b6fb023bb06071a77275d5250042e22b21
Stats: 43 lines in 3 files changed: 11 ins; 15 del; 17 mod
1713: Fix gradle deprecations
Reviewed-by: ihse
-------------
PR: https://git.openjdk.org/skara/pull/1433
More information about the skara-dev
mailing list