search

remkop / picocli

Picocli is a modern framework for building powerful, user-friendly, GraalVM-enabled command line apps with ease. It supports colors, autocompletion, subcommands, and more. In 1 source file so apps can include as source & avoid adding a dependency. Written in Java, usable from Groovy, Kotlin, Scala, etc.
https://picocli.info
Apache License 2.0
4.92k stars 423 forks source link

Too many warnings about "Invalid picocli.usage.width value" #1389

Open xehpuk opened 3 years ago

xehpuk commented 3 years ago

When I e.g.

System.setProperty("picocli.usage.width", "asdf");

and then print the usage help, the output gets spammed:

[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.
[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.
[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.
[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.
[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.
[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.
[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.
[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.
[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.
[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.
[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.
[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.
[picocli WARN] Invalid picocli.usage.width value 'asdf'. Using usage width 80.

One warning should be enough.

remkop commented 3 years ago

Good point. Thank you for raising this!

Will you be able to provide a pull request for this?