groovy-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (GROOVY-8520) Replace commons-cli with picocli in CliBuilder
Date Thu, 19 Apr 2018 01:20:00 GMT

    [ https://issues.apache.org/jira/browse/GROOVY-8520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16443429#comment-16443429
] 

ASF GitHub Bot commented on GROOVY-8520:
----------------------------------------

GitHub user remkop opened a pull request:

    https://github.com/apache/groovy/pull/687

    Groovy-8520 picocli-based CliBuilder 

    Details are in the JIRA: 
    https://issues.apache.org/jira/browse/GROOVY-8520

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/remkop/groovy GROOVY_2_5_X

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/groovy/pull/687.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #687
    
----
commit ecd3060e0b3943feacb1d575f6e78b2e5c5fade0
Author: Remko Popma <remkop@...>
Date:   2018-04-03T12:05:54Z

    GROOVY-8520 initial version of migrating CliBuilder from commons-cli to picocli

commit 16645af5c8f49cb190b72166e1ee2b59fa1fcdf1
Author: Remko Popma <remkop@...>
Date:   2018-04-03T12:07:53Z

    Merge remote-tracking branch 'upstream/GROOVY_2_5_X' into GROOVY_2_5_X

commit 979d2f974b958b109a2e3ea4562a8666de7406c0
Author: Remko Popma <remkop@...>
Date:   2018-04-04T14:53:08Z

    GROOVY-8520 fix failing test CliBuilderTest.testConvertClass

commit c42284afefeff6586e5afe51b3367b84c90fabcf
Author: Remko Popma <remkop@...>
Date:   2018-04-04T14:57:49Z

    GROOVY-8520 fix failing test CliBuilderTest.testParseFromInstance: method invocation parameters
must be an array

commit b9e5ca6c5c1ac0047284812650637435cd462f6e
Author: Remko Popma <remkop@...>
Date:   2018-04-04T15:37:25Z

    GROOVY-8520 cleanup: remove unused code (TypedOption not used any more)

commit 261181a841f6be85b18f54b2cbb31137aaf548ab
Author: Remko Popma <remkop@...>
Date:   2018-04-04T16:10:03Z

    GROOVY-8520 for multi-value options, cli.optionName should return the first element in
the list

commit 69f2660bef3d24a120a64846d1eaa50e3335fc58
Author: Remko Popma <remkop@...>
Date:   2018-04-04T16:33:45Z

    GROOVY-8520 fix failing CliBuilderTest.testMixedBurstingAndLongOptions: to be backwards
compatible, longOpt names should be registered with both a single hyphen and with two hyphens
in picocli.

commit c405cfdd3198e1be73081a5ec6b2395c4149d474
Author: Remko Popma <remkop@...>
Date:   2018-04-04T16:38:01Z

    GROOVY-8520 fix CliBuilderTest.testMixedBurstingAndLongOptions: if you reuse the same
CliBuilder instance multiple times, the previous parse result interacts with the next parse
result (e.g., boolean flags are toggled, collections are added to, etc)

commit bc0005254a50184a24fef337eabae896375787ac
Author: Remko Popma <remkop@...>
Date:   2018-04-04T17:25:00Z

    GROOVY-8520 add back methods to OptionAccessor

commit 5ea2733adf80ef48335d20e99cfe68c846287afd
Author: Remko Popma <remkop@...>
Date:   2018-04-04T18:04:13Z

    GROOVY-8520 remove dependency on commons-cli from CliBuilderTest

commit 7c1530e7d873bfce9e8b3d549f18a2b20a2c1776
Author: Remko Popma <remkop@...>
Date:   2018-04-05T23:19:55Z

    GROOVY-8520 bugfixes, revert TypedOption, support name property, suppress trace output

commit 29dab5f62abb8f1ad67e31e52ca422002d3c9024
Author: Remko Popma <remkop@...>
Date:   2018-04-05T23:21:11Z

    GROOVY-8520 expected output of usage help message is different with picocli

commit d0cf7a2035524ea13249ac1ae2379ee949aaa77c
Author: Remko Popma <remkop@...>
Date:   2018-04-06T14:35:02Z

    GROOVY-8520 replace Unmatched with PositionalParams to support strongly typed @Unparsed
fields/methods

commit 1821e98fb85432bd8fa7ff144105782e62b3113f
Author: Remko Popma <remkop@...>
Date:   2018-04-06T14:37:30Z

    GROOVY-8520 Fix test: should not reuse CliBuilder instance for new input

commit 7668333c8010e2a9d2866a92ad2c75e9b13c820b
Author: Remko Popma <remkop@...>
Date:   2018-04-06T17:02:17Z

    Merge remote-tracking branch 'upstream/GROOVY_2_5_X' into GROOVY_2_5_X

commit c342b2ea9a9a63649415a8b373d62b10fae4d474
Author: Remko Popma <remkop@...>
Date:   2018-04-10T13:08:26Z

    GROOVY-8520 use new picocli-3.0.0-alpha-5 API, fix many bugs

commit e6652b9acdbf049fae1ef91a013db371d27bfee5
Author: Remko Popma <remkop@...>
Date:   2018-04-10T18:25:34Z

    GROOVY-8520 fix failing test

commit 6a913e0dec8adbaf72b1a7836599be51061d2b1a
Author: Remko Popma <remkop@...>
Date:   2018-04-10T18:27:38Z

    GROOVY-8520 create new CliBuilder instance before parsing

commit 699521b019527b38bf1914949d8d8da211ec28af
Author: Remko Popma <remkop@...>
Date:   2018-04-10T18:28:38Z

    GROOVY-8520 create new CliBuilder instance before parsing

commit eb5aab2b0bdbe3b125f5b19573f631554c430891
Author: Remko Popma <remkop@...>
Date:   2018-04-10T18:33:18Z

    Merge remote-tracking branch 'upstream/GROOVY_2_5_X' into GROOVY_2_5_X

commit 98dbd9108d1f543a56ef573af8ea4c175121f2f2
Author: Remko Popma <remkop@...>
Date:   2018-04-10T18:34:06Z

    GROOVY-8520 bump picocliVersion to v3.0.0-alpha-5

commit 29174a7c0bc99a7d0491aa68f1d0e1f0c91dc5a2
Author: Remko Popma <remkop@...>
Date:   2018-04-12T13:52:59Z

    GROOVY-8520 no need to create new CliBuilder instance before parsing anymore

commit 82a6f38533a3943c1909392a299929aaceb20ec8
Author: Remko Popma <remkop@...>
Date:   2018-04-12T13:54:58Z

    GROOVY-8520 no need to create new CliBuilder instance before parsing anymore

commit b22ce49cc47085a1990f13e4d1ea296d62130892
Author: Remko Popma <remkop@...>
Date:   2018-04-12T13:56:12Z

    GROOVY-8520 no need to create new CliBuilder instance before parsing anymore

commit 0d82aa9ba040bc18f9ba78aa32256a872f9aafcf
Author: Remko Popma <remkop@...>
Date:   2018-04-12T14:49:13Z

    GROOVY-8520 bump picocliVersion to v3.0.0-alpha-6

commit 251d31922bc1765d79790d79d72aa0f660a839b4
Author: Remko Popma <remkop@...>
Date:   2018-04-12T14:57:05Z

    Merge remote-tracking branch 'upstream/GROOVY_2_5_X' into GROOVY_2_5_X

commit 55b523473e1b446062036f123542040868ec78c3
Author: Remko Popma <remkop@...>
Date:   2018-04-12T22:39:50Z

    GROOVY-8520 fix for testWithArgument

commit d1211ad24dd37dc85418ec60d579047364ce42a5
Author: Remko Popma <remkop@...>
Date:   2018-04-13T14:40:51Z

    GROOVY-8520 docs; fix initialization bug on reuse (requires 3.0.0-beta-1 ArgSpec.Builder.hasInitialValue
API)

commit eb12d2c69f08db85dce5594737b659cf9d1e856f
Author: Remko Popma <remkop@...>
Date:   2018-04-13T17:02:33Z

    GROOVY-8520 docs; fix NPE when usage() called before parse()

commit 8317bcf011df40399db229553faa12ffd6ffe979
Author: Remko Popma <remkop@...>
Date:   2018-04-13T17:13:20Z

    GROOVY-8520 docs

----


> Replace commons-cli with picocli in CliBuilder
> ----------------------------------------------
>
>                 Key: GROOVY-8520
>                 URL: https://issues.apache.org/jira/browse/GROOVY-8520
>             Project: Groovy
>          Issue Type: Improvement
>          Components: command line processing
>            Reporter: Remko Popma
>            Priority: Major
>
> This ticket proposes to replace commons-cli with picocli in {{groovy.util.CliBuilder}}.
> See [discussion on the mailing list|https://lists.apache.org/thread.html/d60b6d5d4411e9ba0d7dc209cde8a9bb4abb00f0b9c0322f068c322e@%3Cdev.groovy.apache.org%3E]
for the original proposal and comparison with other CLI libraries.
> Goals for the initial implementation:
> * preserve the current CliBuilder behaviour as much as possible
> * deliver an implementation, tests and documentation in time to be included in the 2.5
GA release



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message