Github user twogee commented on a diff in the pull request:
https://github.com/apache/ant-ivy/pull/71#discussion_r177857645
--- Diff: src/java/org/apache/ivy/Main.java ---
@@ -199,6 +201,10 @@ static CommandLineParser getParser() {
new OptionBuilder("cp").ar
Github user twogee commented on a diff in the pull request:
https://github.com/apache/ant-ivy/pull/71#discussion_r177696348
--- Diff: src/java/org/apache/ivy/Main.java ---
@@ -199,6 +201,10 @@ static CommandLineParser getParser() {
new OptionBuilder("cp").ar
Github user jaikiran commented on a diff in the pull request:
https://github.com/apache/ant-ivy/pull/71#discussion_r177694759
--- Diff: src/java/org/apache/ivy/Main.java ---
@@ -199,6 +201,10 @@ static CommandLineParser getParser() {
new OptionBuilder("cp").
Github user jaikiran commented on a diff in the pull request:
https://github.com/apache/ant-ivy/pull/71#discussion_r177694862
--- Diff: src/java/org/apache/ivy/Main.java ---
@@ -199,6 +201,10 @@ static CommandLineParser getParser() {
new OptionBuilder("cp").
Github user jaikiran commented on the issue:
https://github.com/apache/ant-ivy/pull/71
@aanno Thank you for the patch. This mostly looks fine. I have added some
review comments. Can you please also introduce a test case for this? Something
that tests that these command options are rec