deprecate buildNative gradle task

[glefloch]

this pull request deprecates gradle buildNative task.

close #7104

[gastaldi]

LGTM

[gastaldi]

You need to update the io.quarkus.gradle.QuarkusPluginFunctionalTest too I suppose

[gastaldi]

Also make sure the Gradle tests pass in your environment by running gradle test functionalTest in the plugin folder

[glefloch]

Thank you for the review, I'm checking why the build fails.

[machi1990]

Thanks for the contribution @glefloch, can we squash the commits into one?

[glefloch]

Yes, for sure, I squashed commits.

[gastaldi]

Added some comments below. Can you verify them before we merge it?

[gastaldi]

Isn't this already done in configureBuildNativeTask(...)?

[glefloch]

well, yes, you right, I moved out the logger and used task path instead of task reference.

[gastaldi]

It seems that the warning shows only after quarkusBuild executes. Is there any way to show that warning BEFORE quarkusBuild kicks in?

[gastaldi]

Also, don't forget to update the READMEs generated when a project is created:
https://github.com/quarkusio/quarkus/blob/master/devtools/platform-descriptor-json/src/main/resources/templates/README.gradle.ftl#L27-L31

[glefloch]

It seems that the warning shows only after quarkusBuild executes. Is there any way to show that warning BEFORE quarkusBuild kicks in?

Well, we could either move the warning in the configureBuildNative method, or instead of making buildNative depends on quarkusBuild, we could set buildNative to be finalized by quarkusBuild. What do you think @gastaldi ?

[gastaldi]

I am not sure which is better. I'll let you decide 😉

[glefloch]

@gastaldi, I updated buildNative task to be finalizedBy by quarkusBuild. The output looks like:

> Task :buildNative
buildNative task has been deprecated in favor of quarkusBuild -Dquarkus.package.type=native

> Task :quarkusBuild
building quarkus runner

BUILD SUCCESSFUL in 1m 20s

WDYT?