[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15338177#comment-15338177 ] ASF subversion and git services commented on GEODE-835: --- Commit 947628a202f8caa197fd2911257bef0dd754a201 in incubator-geode's branch refs/heads/feature/GEODE-1565 from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=947628a ] GEODE-835: remove developer name from comments > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15336870#comment-15336870 ] ASF subversion and git services commented on GEODE-835: --- Commit 947628a202f8caa197fd2911257bef0dd754a201 in incubator-geode's branch refs/heads/develop from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=947628a ] GEODE-835: remove developer name from comments > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15334792#comment-15334792 ] ASF subversion and git services commented on GEODE-835: --- Commit 921bec11d89866e3332fd3e7ce2ac2f9e8526eb1 in incubator-geode's branch refs/heads/feature/GEODE-420 from [~gmeilen] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=921bec1 ] GEODE-835: replace geode-joptsimple with jopt-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 * This closes #162 [kl...@apache.org] > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15334351#comment-15334351 ] ASF subversion and git services commented on GEODE-835: --- Commit 1c5ba1415299313fddc8d9d11396f0b995a2b12a in incubator-geode's branch refs/heads/develop from [~gmeilen] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=1c5ba14 ] GEODE-835: replace geode-joptsimple with jopt-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 * This closes #162 [kl...@apache.org] > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15334354#comment-15334354 ] ASF GitHub Bot commented on GEODE-835: -- Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15334221#comment-15334221 ] ASF subversion and git services commented on GEODE-835: --- Commit fab62c21cd375da550c8e54da1bc74c7891c17d9 in incubator-geode's branch refs/heads/feature/GEODE-835 from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=fab62c2 ] Merge remote-tracking branch 'origin/develop' into feature/GEODE-835 > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15325355#comment-15325355 ] ASF subversion and git services commented on GEODE-835: --- Commit 0ad4acef15aca9d9d2ddf4f44ea47a8907df4838 in incubator-geode's branch refs/heads/feature/GEODE-835 from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=0ad4ace ] Merge remote-tracking branch 'origin/develop' into feature/GEODE-835 > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15323097#comment-15323097 ] ASF subversion and git services commented on GEODE-835: --- Commit 347064abc5b06eabd35ee6caa33f62115de3263d in incubator-geode's branch refs/heads/feature/GEODE-835 from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=347064a ] Merge remote-tracking branch 'origin/develop' into feature/GEODE-835 > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15319744#comment-15319744 ] ASF subversion and git services commented on GEODE-835: --- Commit c66252eb1f7a55ea0b953cf53b098862e27dfb51 in incubator-geode's branch refs/heads/feature/GEODE-835 from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=c66252e ] Merge remote-tracking branch 'origin/develop' into feature/GEODE-835 > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15301009#comment-15301009 ] ASF subversion and git services commented on GEODE-835: --- Commit c9aac5a335ffc0c41e80f26b1cf194765613118d in incubator-geode's branch refs/heads/feature/GEODE-835 from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=c9aac5a ] Merge remote-tracking branch 'origin/develop' into feature/GEODE-835 > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15300954#comment-15300954 ] Kirk Lund commented on GEODE-835: - The two tests closest to the jopt-simple OptionParser code are JoptOptionParserTest and GfshParserJUnitTest. > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15294564#comment-15294564 ] ASF subversion and git services commented on GEODE-835: --- Commit ce2ac68308448309f5501ce9683b91d2b1e74630 in incubator-geode's branch refs/heads/feature/GEODE-835 from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=ce2ac68 ] Merge remote-tracking branch 'origin/develop' into feature/GEODE-835 > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15294563#comment-15294563 ] ASF subversion and git services commented on GEODE-835: --- Commit 853c40d26fe63f2411fb735b263efee11afdfa1a in incubator-geode's branch refs/heads/feature/GEODE-835 from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=853c40d ] GEODE-835: Create JoptOptionParserTest JoptOptionParser is the primary geode class using joptsimple so this test is necessary before replacing geode-joptsimple with a dependency on jopt-simple 5.0.1. * cleanup CLI exceptions and the constructors for these exceptions > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15294558#comment-15294558 ] ASF subversion and git services commented on GEODE-835: --- Commit 853c40d26fe63f2411fb735b263efee11afdfa1a in incubator-geode's branch refs/heads/develop from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=853c40d ] GEODE-835: Create JoptOptionParserTest JoptOptionParser is the primary geode class using joptsimple so this test is necessary before replacing geode-joptsimple with a dependency on jopt-simple 5.0.1. * cleanup CLI exceptions and the constructors for these exceptions > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15294229#comment-15294229 ] ASF subversion and git services commented on GEODE-835: --- Commit 52f21b0fce4fd259c05b30c1b8efac04588819d4 in incubator-geode's branch refs/heads/feature/GEODE-835 from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=52f21b0 ] Merge remote-tracking branch 'origin/develop' into feature/GEODE-835 > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293747#comment-15293747 ] ASF subversion and git services commented on GEODE-835: --- Commit 7e559224ad24789f288f1ef02c3b6029ceee5547 in incubator-geode's branch refs/heads/feature/GEODE-835 from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=7e55922 ] Merge remote-tracking branch 'origin/develop' into feature/GEODE-835 > 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)
[jira] [Commented] (GEODE-835) Replace joptsimple source with a binary dependency
[ https://issues.apache.org/jira/browse/GEODE-835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293660#comment-15293660 ] ASF subversion and git services commented on GEODE-835: --- Commit 45b42477805ac9f08deb33fda357e6f6a5fe7b1b in incubator-geode's branch refs/heads/feature/GEODE-835-test from [~apa...@the9muses.net] [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=45b4247 ] Merge remote-tracking branch 'origin/develop' into feature/GEODE-835-test > 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)
