Skip to content
This repository has been archived by the owner on Feb 23, 2023. It is now read-only.

Review Maven plugin logging #491

Closed
sdeleuze opened this issue Feb 18, 2021 · 3 comments
Closed

Review Maven plugin logging #491

sdeleuze opened this issue Feb 18, 2021 · 3 comments
Assignees
Labels
type: bug A general bug
Milestone

Comments

@sdeleuze
Copy link
Contributor

It seems also that System.out.println are not working anymore in ConfigOptions after the switch to bti, so we should maybe use a logger to get the relevant output like for System.out.println("Spring Native operating in "+MODE+" mode");.

@sdeleuze sdeleuze added this to the 0.9.0 milestone Feb 18, 2021
@sdeleuze sdeleuze added the type: bug A general bug label Feb 18, 2021
@aclement
Copy link
Contributor

All the existing verbose logging with spring.native.verbose was converted to proper Java logging. I'll change the remaining few sysouts under this bug. You can turn on verbose mode by using the standard maven -X option which turns on debug logging for a run (you will see everything that previously came out in verbose mode).

@sdeleuze sdeleuze changed the title Fix ConfigOptions (and potentially other places) logging Fix Maven plugin logging Feb 19, 2021
@sdeleuze sdeleuze self-assigned this Feb 19, 2021
@sdeleuze
Copy link
Contributor Author

I reopen it to review logging once #484 will be fixed since I can't see logs for example Spring Native operating in ... mode which was displayed without having to set spring.native.verbose. I think we should avoid requiring having to set -X, -e should be enough.

@sdeleuze sdeleuze reopened this Feb 19, 2021
@sdeleuze sdeleuze changed the title Fix Maven plugin logging Review Maven plugin logging Feb 19, 2021
@sdeleuze
Copy link
Contributor Author

With 78f792c it seems to work properly, I have configured Spring Native operating in ... mode to logged at info level.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug A general bug
Development

No branches or pull requests

2 participants