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

ASF GitHub Bot commented on GEODE-835:
--------------------------------------

GitHub user gracemeilen opened a pull request:

    https://github.com/apache/incubator-geode/pull/162

    GEODE-835: replace geode-joptsimple with jopt-simple dependency

    GEODE-835: replace geode-joptsimple with jolt-simple dependency
    
    * remove geode-joptsimple module
    * add dependency on jopt-simple 5.0.1
    * wrap value of --J option with quotes
    * create tests OptionJFormatterTest, CommentSkipHelperTest, 
GfshParserIntegrationTest
    * invalid arguments are silently ignored

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

    $ git pull https://github.com/gracemeilen/incubator-geode develop

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

    https://github.com/apache/incubator-geode/pull/162.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 #162
    
----

----


> Replace joptsimple source with a binary dependency
> --------------------------------------------------
>
>                 Key: GEODE-835
>                 URL: https://issues.apache.org/jira/browse/GEODE-835
>             Project: Geode
>          Issue Type: Bug
>          Components: build, gfsh
>            Reporter: Anthony Baker
>            Assignee: Kirk Lund
>             Fix For: 1.0.0-incubating.M3
>
>         Attachments: joptsimple.patch
>
>
> The gemfire-jopsimple folder contains modified source code from the 
> joptsimple library.  We should replace this with a binary dependency.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to