remove redundant configuration loading for live dev mode

[johnaohara]

Currently mvn quarkus:dev only reads properties from "META-INF/microprofile-config.properties". The maven plugin does not look for properties in "application.properties", so any properties defined in "application.properties" will not be loaded in live dev mode

[geoand]

This should have been fixed in: #1249. Are you sure you are using the latest release?

[johnaohara]

I was looking here;

https://github.com/quarkusio/quarkus/blob/master/core/devmode/src/main/java/io/quarkus/dev/DevModeMain.java#L62

Are the property files resolved in multiple locations?

[geoand]

To tell you the truth I don't know that part of the code is supposed to do.

But the files to be config to be checked for change are consulted at:
https://github.com/quarkusio/quarkus/blob/master/core/devmode/src/main/java/io/quarkus/dev/RuntimeUpdatesProcessor.java#L149
https://github.com/quarkusio/quarkus/blob/master/core/devmode/src/main/java/io/quarkus/dev/HotDeploymentConfigFileBuildStep.java#L35

Do you have an example that you ran and the config change did not get picked up? The simple examples I tried worked fine

[johnaohara]

I haven't run an example, I was trawling through the code and noticed that it was not searching for application.properties. I will double check if this causes an issue somewhere

[geoand]

Cool, thanks

[johnaohara]

@geoand I have just spoken to @stuartwdouglas . The code at https://github.com/quarkusio/quarkus/blob/master/core/devmode/src/main/java/io/quarkus/dev/DevModeMain.java#L62 might be redundant. I am testing a patch now

[geoand]

Sounds good :)