RFR: 6946: There should be an option to build the agent in the build script [v20]

[Alex Macdonald]

On Wed, 13 Jan 2021 15:02:16 GMT, Miroslav Wengner <mwengner at openjdk.org> wrote:

>> https://bugs.openjdk.java.net/browse/JMC-6946
>> added: 
>> - build.sh --packageAgent
>> - build.sh --runAgentExample
>
> Miroslav Wengner has updated the pull request incrementally with one additional commit since the last revision:
> 
>   6946: corrected typo

Just a couple of minor nits, the functionality seems fine to me.

build.sh line 83:

> 81:         printf " \t%s\t%s\n" "--test" "to run the tests"
> 82:         printf " \t%s\t%s\n" "--testUi" "to run the tests including UI tests"
> 83:         printf " \t%s\t%s\n" "--installCore" "install JMC core"

If trying to keep consistency, the `--installCore` is missing the "to" in it's description; it has it in the .bat file though

build.sh line 88:

> 86:         printf " \t%s\t%s\n" "--clean" "to run maven clean"
> 87:         printf " \t%s\t%s\n" "--run" "to run JMC once it was packaged"
> 88:         printf " \t%s\t%s\n" "--runAgentExample" "to run Agent 'InstrumentMe' example, once it was packaged"

For `--run`, `--runAgentExample`, and `--runAgentConverterExample`, I think it reads better for the description to say "once it _is_ packaged" rather than "was"

-------------

PR: https://git.openjdk.java.net/jmc/pull/183