[GitHub] commons-cli issue #16: added findbug and checkstyle support while building

2017-08-02 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue: https://github.com/apache/commons-cli/pull/16 The findbugs version used is not java 6 compatible. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] commons-cli issue #16: added findbug and checkstyle support while building

2017-08-02 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue: https://github.com/apache/commons-cli/pull/16 The findbugs version used is not java 6 compatible. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] commons-text pull request #61: Added flexible Travis-ci build script.

2017-08-02 Thread ameyjadiye
Github user ameyjadiye commented on a diff in the pull request: https://github.com/apache/commons-text/pull/61#discussion_r130978503 --- Diff: pom.xml --- @@ -171,6 +171,7 @@ src/site/resources/download_lang.cgi

[GitHub] commons-cli issue #16: added findbug and checkstyle support while building

2017-08-02 Thread ameyjadiye
Github user ameyjadiye commented on the issue: https://github.com/apache/commons-cli/pull/16 @PascalSchumacher , can you take a look at this ? seems only issue with findbug on java6 build, rest seems Ok. --- If your project is set up for it, you can reply to this email and have

[GitHub] commons-cli issue #16: added findbug and checkstyle support while building

2017-08-02 Thread ameyjadiye
Github user ameyjadiye commented on the issue: https://github.com/apache/commons-cli/pull/16 @PascalSchumacher , can you take a look at this ? seems only issue with findbug on java6 build, rest seems Ok. --- If your project is set up for it, you can reply to this email and have

[GitHub] commons-text pull request #61: Added flexible Travis-ci build script.

2017-08-02 Thread chtompki
Github user chtompki commented on a diff in the pull request: https://github.com/apache/commons-text/pull/61#discussion_r130979086 --- Diff: pom.xml --- @@ -171,6 +171,7 @@ src/site/resources/download_lang.cgi

[GitHub] commons-text issue #61: Added flexible Travis-ci build script.

2017-08-02 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue: https://github.com/apache/commons-text/pull/61 Also note that currently this script only compiles and runs test. Rat, clirr, findbugs and javadoc are not executed. --- If your project is set up for it, you can reply to this email

[GitHub] commons-text pull request #61: Added flexible Travis-ci build script.

2017-08-02 Thread chtompki
Github user chtompki commented on a diff in the pull request: https://github.com/apache/commons-text/pull/61#discussion_r130977723 --- Diff: pom.xml --- @@ -171,6 +171,7 @@ src/site/resources/download_lang.cgi

Re: [all/travis-ci] Regarding using ibmjava on travis

2017-08-02 Thread Amey Jadiye
Hello All, We need community opinion here as this change request can improve / affect many other components in near future. I have proposed the change in PR https://github.com/apache/commons-text/pull/61 so the travis builds can become more flexible and we will have more control over the

Re: [VOTE] Release Apache Commons JCS 2.2 based on RC1

2017-08-02 Thread Thomas Vandahl
On 02.08.17 15:32, Thomas Vandahl wrote: > [X] +1 Release these artifacts > [ ] +0 OK, but... > [ ] -0 OK, but really should fix... > [ ] -1 I oppose this release because... > Bye, Thomas - To unsubscribe, e-mail:

[GitHub] commons-text issue #61: Added flexible Travis-ci build script.

2017-08-02 Thread chtompki
Github user chtompki commented on the issue: https://github.com/apache/commons-text/pull/61 Personally I'm a +0 to +1 on this, but @PascalSchumacher has a very valid point. I try to subscribe to the thought that there should be a high bar to sacrificing simplicity. So, with that in

[VOTE] Release Apache Commons JCS 2.2 based on RC1

2017-08-02 Thread Thomas Vandahl
I would like to release the [jcs] component to resolve some overdue bugs Apache Commons JCS 2.2 RC1 is available for review at: https://dist.apache.org/repos/dist/dev/commons/jcs/ (r20728). Maven artifacts are at: https://repository.apache.org/content/repositories/orgapachecommons-1256 .

[GitHub] commons-io pull request #41: Missing Test Case for negative offset value

2017-08-02 Thread ileontiuc
GitHub user ileontiuc opened a pull request: https://github.com/apache/commons-io/pull/41 Missing Test Case for negative offset value The method copyLarge(InputStream input, InputStream output, int offset, buffer) copies the content of input into output. If offset is a positive

Re: [VOTE] Release Apache Commons JCS 2.2 based on RC1

2017-08-02 Thread Romain Manni-Bucau
+1, thanks Thomas Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github | LinkedIn | JavaEE Factory

[GitHub] commons-io pull request #41: Missing Test Case for negative offset value

2017-08-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-io/pull/41 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] commons-text issue #61: Added flexible Travis-ci build script.

2017-08-02 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue: https://github.com/apache/commons-text/pull/61 +1 for moving this discussion to the mailing list --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does