Re: [VOTE] Release Commons Fileupload 1.3.3 based on RC6

2017-06-13 Thread Rob Tompkins
Here’s my implicit +1 (non-binding) vote. > On Jun 9, 2017, at 9:57 PM, Rob Tompkins wrote: > > Hello all, > > This is a [VOTE] for releasing Apache Commons Fileupload 1.3.3 (from RC6). > > Tag name: > commons-fileupload-1.3.3-RC6 (signature can be checked from git

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-13 Thread schaumb
Github user schaumb commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121654518 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-13 Thread schaumb
Github user schaumb commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121654518 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli issue #12: [CLI-274] implement EXISTING_FILE_VALUE type handler

2017-06-13 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-cli/pull/12 [![Coverage Status](https://coveralls.io/builds/11947210/badge)](https://coveralls.io/builds/11947210) Coverage increased (+0.009%) to 96.244% when pulling

[GitHub] commons-cli issue #12: [CLI-274] implement EXISTING_FILE_VALUE type handler

2017-06-13 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-cli/pull/12 [![Coverage Status](https://coveralls.io/builds/11947210/badge)](https://coveralls.io/builds/11947210) Coverage increased (+0.009%) to 96.244% when pulling

Re: [DRAFT] Apache Commons Board Report for June 2017

2017-06-13 Thread Rob Tompkins
> On Jun 13, 2017, at 2:05 AM, Benedikt Ritter wrote: > > Hello Gary, > >> Am 13.06.2017 um 03:23 schrieb Gary Gregory : >> >> Any and all feedback is welcome. I would like to send the report in 24 >> hours. >> >> ## Description: >> >> - The

[RESULT][VOTE] Release Commons Fileupload 1.3.3 based on RC6

2017-06-13 Thread Rob Tompkins
The vote passes with the following votes (in order of appearance): Gary Gregory: +1 Bruno P. Kinoshita: +1 Oliver Heger: +1 Rob Tompkins: +1 (non-binding) I will proceed with publishing the release candidate. Many thanks to all of those that helped in preparing and validating this release.

Re: [fileupload] Please add your release data for 'commons'

2017-06-13 Thread Gilles
Hi Rob. On Tue, 13 Jun 2017 08:15:59 -0400, Rob Tompkins wrote: On Jun 13, 2017, at 8:14 AM, Apache Reporter Service wrote: Hi, This is an automated email from reporter.apache.org. I see that you just pushed something to our release repository for the 'commons'

Re: [fileupload] Please add your release data for 'commons'

2017-06-13 Thread Rob Tompkins
> On Jun 13, 2017, at 8:22 AM, Gilles wrote: > > Hi Rob. > > On Tue, 13 Jun 2017 08:15:59 -0400, Rob Tompkins wrote: >>> On Jun 13, 2017, at 8:14 AM, Apache Reporter Service >>> wrote: >>> >>> Hi, >>> This is an automated email

[Numbers] Methods removal (in "Precision" class)

2017-06-13 Thread Gilles
Hi. Please raise any concerns concerning this proposal: https://issues.apache.org/jira/browse/NUMBERS-43 Regards, Gilles - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail:

[fileupload] Please add your release data for 'commons'

2017-06-13 Thread Rob Tompkins
> On Jun 13, 2017, at 8:14 AM, Apache Reporter Service > wrote: > > Hi, > This is an automated email from reporter.apache.org. > I see that you just pushed something to our release repository for the > 'commons' project > in the following commit: > > r20017 at

[GitHub] commons-compress issue #31: COMPRESS-400 It should be possible for users to ...

2017-06-13 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/31 [![Coverage Status](https://coveralls.io/builds/11955168/badge)](https://coveralls.io/builds/11955168) Coverage decreased (-0.02%) to 84.322% when pulling

[GitHub] commons-compress issue #32: Enable maven cache for in travis config

2017-06-13 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/32 [![Coverage Status](https://:/builds/11956920/badge)](https://:/builds/11956920) Coverage remained the same at 84.339% when pulling

[GitHub] commons-compress pull request #33: add-some-Unit-Tests Added some Unit Tests...

2017-06-13 Thread TheRealHaui
GitHub user TheRealHaui opened a pull request: https://github.com/apache/commons-compress/pull/33 add-some-Unit-Tests Added some Unit Tests to increase code coverage. I have added some Unit Tests to increase code coverage and want to contribute them. You can merge this pull

Re: [jelly] Build for tag lib projects

2017-06-13 Thread Benedikt Ritter
Sounds good to me. Oliver Heger schrieb am Di. 13. Juni 2017 um 21:02: > Hi, > > so far the Jelly tag libs are not integrated into the main build and do > not support Maven 3. I think, this should be changed, and the build > should be transformed to a multi-module

[GitHub] commons-compress pull request #32: Enable maven cache for in travis config

2017-06-13 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/32 Enable maven cache for in travis config Signed-off-by: Simon Spero You can merge this pull request into a Git repository by running: $ git pull

Re: [jelly] Support for multiple build tools

2017-06-13 Thread Amey Jadiye
+1 Regards, Amey On Wed, Jun 14, 2017, 12:22 AM Oliver Heger wrote: > Hi, > > locally I reworked the build of Jelly core to follow the Maven standard > layout. This of course renders all other build files dysfunctional. > > Are there any objections to drop these

Re: OSGI Version at Package Level : some more details about the COMPRESS-399 pull request

2017-06-13 Thread Simon Spero
On Jun 13, 2017 1:48 PM, "Gary Gregory" wrote: I think that the best way to see how this works is to provide a patch... then we can see how it goes. Gary- Can you clarify a few things: What would you like to see patches for - the commons parent / related mojo one or

Re: [jelly] Support for multiple build tools

2017-06-13 Thread Benedikt Ritter
+1, drop that stuff! Oliver Heger schrieb am Di. 13. Juni 2017 um 20:52: > Hi, > > locally I reworked the build of Jelly core to follow the Maven standard > layout. This of course renders all other build files dysfunctional. > > Are there any objections to drop

[GitHub] commons-compress issue #33: add-some-Unit-Tests Added some Unit Tests to inc...

2017-06-13 Thread TheRealHaui
Github user TheRealHaui commented on the issue: https://github.com/apache/commons-compress/pull/33 @bodewig Could you be so kind an review my contribution and - of course - most of all pull it into the code base. --- If your project is set up for it, you can reply to this email

Re: [jelly] Support for multiple build tools

2017-06-13 Thread Bruno P. Kinoshita
+1 From: Benedikt Ritter To: Commons Developers List Sent: Wednesday, 14 June 2017 8:04 AM Subject: Re: [jelly] Support for multiple build tools +1, drop that stuff! Oliver Heger

[GitHub] commons-compress issue #31: COMPRESS-400 It should be possible for users to ...

2017-06-13 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/31 [![Coverage Status](https://:/builds/11956800/badge)](https://:/builds/11956800) Coverage increased (+0.1%) to 84.458% when pulling

[GitHub] commons-compress issue #33: add-some-Unit-Tests Added some Unit Tests to inc...

2017-06-13 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/33 [![Coverage Status](https://:/builds/11957412/badge)](https://:/builds/11957412) Coverage increased (+0.2%) to 84.524% when pulling

[GitHub] commons-compress issue #31: COMPRESS-400 It should be possible for users to ...

2017-06-13 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/31 [![Coverage Status](https://:/builds/11957185/badge)](https://:/builds/11957185) Coverage increased (+0.1%) to 84.458% when pulling

[GitHub] commons-compress issue #31: COMPRESS-400 It should be possible for users to ...

2017-06-13 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/31 [![Coverage Status](https://:/builds/11957185/badge)](https://:/builds/11957185) Coverage increased (+0.1%) to 84.458% when pulling

[GitHub] commons-compress issue #31: COMPRESS-400 It should be possible for users to ...

2017-06-13 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/31 [![Coverage Status](https://:/builds/11957185/badge)](https://:/builds/11957185) Coverage increased (+0.1%) to 84.458% when pulling

Re: [DRAFT] Apache Commons Board Report for June 2017

2017-06-13 Thread Gary Gregory
On Mon, Jun 12, 2017 at 11:05 PM, Benedikt Ritter wrote: > Hello Gary, > > > Am 13.06.2017 um 03:23 schrieb Gary Gregory : > > > > Any and all feedback is welcome. I would like to send the report in 24 > > hours. > > > > ## Description: > > > > - The

Re: [DRAFT] Apache Commons Board Report for June 2017

2017-06-13 Thread Gary Gregory
Another thing I'll mention: - We are starting to add testing with IBM Java 8 to Travis-CI builds. Gary On Mon, Jun 12, 2017 at 11:28 PM, Gary Gregory wrote: > > On Mon, Jun 12, 2017 at 11:05 PM, Benedikt Ritter > wrote: > >> Hello Gary, >> >> > Am

Re: [DRAFT] Apache Commons Board Report for June 2017

2017-06-13 Thread Benedikt Ritter
Hello Gary, > Am 13.06.2017 um 03:23 schrieb Gary Gregory : > > Any and all feedback is welcome. I would like to send the report in 24 > hours. > > ## Description: > > - The Apache Commons project focuses on all aspects of reusable Java > components. > > - The Apache

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-13 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121588869 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-13 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121589195 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-13 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121589022 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-13 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121589022 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-13 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121589195 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-13 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121588869 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

Re: OSGI Version at Package Level : some more details about the COMPRESS-399 pull request

2017-06-13 Thread Gary Gregory
I think that the best way to see how this works is to provide a patch... then we can see how it goes. Gary On Mon, Jun 12, 2017 at 10:18 AM, Simon Spero wrote: > On Mon, Jun 12, 2017 at 10:53 AM, Matt Sicker wrote: > > > I'd love to have a semantic

[jelly] Support for multiple build tools

2017-06-13 Thread Oliver Heger
Hi, locally I reworked the build of Jelly core to follow the Maven standard layout. This of course renders all other build files dysfunctional. Are there any objections to drop these files? Maven 1 is probably no longer in use anywhere. Most other Commons components have already dropped their

[jelly] Build for tag lib projects

2017-06-13 Thread Oliver Heger
Hi, so far the Jelly tag libs are not integrated into the main build and do not support Maven 3. I think, this should be changed, and the build should be transformed to a multi-module project that builds core and the tag libs as well. (Although it may make sense here that single tag lib projects

Re: [DRAFT] Apache Commons Board Report for June 2017

2017-06-13 Thread Gary Gregory
On Tue, Jun 13, 2017 at 4:56 AM, Rob Tompkins wrote: > > > On Jun 13, 2017, at 2:05 AM, Benedikt Ritter wrote: > > > > Hello Gary, > > > >> Am 13.06.2017 um 03:23 schrieb Gary Gregory : > >> > >> Any and all feedback is welcome. I

[GitHub] commons-text issue #48: TEXT-88: Fixed issue with empty delimiters array wit...

2017-06-13 Thread arunvinudss
Github user arunvinudss commented on the issue: https://github.com/apache/commons-text/pull/48 @ameyjadiye The agreed upon fix as of now should be: WordUtils.capitalizeFully("i am fine", new char[]{}) --> I am fine We shouldn't take space as the delimiter as the user