Make sure ApplicationConfig uses the correct resource dir

[stalep]

No description provided.

[geoand]

I admit I have missed the recent gradle changes but I am still not sure if this will work correctly, because the "reading" part of the code is: https://github.com/quarkusio/quarkus/blob/master/core/deployment/src/main/java/io/quarkus/deployment/steps/ApplicationInfoBuildStep.java#L47

Now that I think of it, maybe the change isn't even needed and things will work as is?

Let me try a sample gradle project and get back to you

[geoand]

So I tried the gradle app without the PR and the new BuildItem (ApplicationInfoBuildItem) added in #1683 works just fine. With the change here it does not work (basically the defaults of name and version are not properly set), I guess because the Classloader on the reading side does not know where to find the resource.

In any case, this properties file is strictly internal, it should never be even be seen by users and is only used in order to populate a BuildItem.
Furthermore, this is a temporary solution until #1693 comes up with a holistic approach

[stalep]

hm, ok. ill close this, i just thought it should continue to use the default resource folder defined by each build system. lets discuss this further in #1693