Re: [VOTE] Release Apache Commons Lang 3.6 based on RC3

2017-06-08 Thread Gary Gregory
On Thu, Jun 8, 2017 at 6:29 PM, Simon Spero wrote: > There is a one other compatibility issue, which can be seen in the attached > code: > > import java.nio.charset.StandardCharsets; > > public class Weasel { > > private static final String US_ASCII = "US-ASCII"; >

Re: [VOTE] Release Apache Commons Lang 3.6 based on RC3

2017-06-08 Thread Simon Spero
There is a one other compatibility issue, which can be seen in the attached code: import java.nio.charset.StandardCharsets; public class Weasel { private static final String US_ASCII = "US-ASCII"; private static final String UTF_8 = "UTF-8"; private static final String

Re: NUMBERS-40: Review exception usage in package "o.a.c.numbers.gamma"

2017-06-08 Thread Tharaka De Silva
Thank you for the response! On Thu, Jun 8, 2017 at 5:13 AM, Gilles wrote: > Hi. > > On Wed, 7 Jun 2017 17:00:46 -0500, Tharaka De Silva wrote: > >> Hello everyone, >> >> I am new to the ASF community >> > > Welcome. Thank you! > > > and decided to grab

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

2017-06-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-cli/pull/10 --- 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-cli pull request #3: [CLI-253] Prevent "Unrecognized option: --null"...

2017-06-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-cli/pull/3 --- 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

Re: [VOTE] Release Apache Commons Lang 3.6 based on RC3

2017-06-08 Thread sebb
On 8 June 2017 at 18:09, Gary Gregory wrote: > On Thu, Jun 8, 2017 at 9:57 AM, sebb wrote: > >> On 8 June 2017 at 17:19, Gary Gregory wrote: >> > On Thu, Jun 8, 2017 at 5:38 AM, Simon Spero wrote: >> > >> >>

Re: [LANG] Fix date related test failures on IBM JDKs (Was: Re: [CANCEL][VOTE] Release Apache Commons Lang 3.6 based on RC2)

2017-06-08 Thread Amey Jadiye
Hi Gary, I have raised PR[1] for this please review, I have already tested on forked repo [2], works like a charm, now each commit will be tested against. - openjdk7 - oraclejdk7 - oraclejdk8 - ibmjdk8 [1] https://github.com/apache/commons-text/pull/45 [2]

[GitHub] commons-text issue #45: Added IBM Jdk8 build support to travis-ci configurat...

2017-06-08 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-text/pull/45 [![Coverage Status](https://coveralls.io/builds/11892865/badge)](https://coveralls.io/builds/11892865) Coverage remained the same at 96.653% when pulling

[GitHub] commons-text issue #45: Added IBM Jdk8 build support to travis-ci configurat...

2017-06-08 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-text/pull/45 [![Coverage Status](https://coveralls.io/builds/11892865/badge)](https://coveralls.io/builds/11892865) Coverage remained the same at 96.653% when pulling

Re: [ANNOUNCE] Apache Commons CLI moved to git

2017-06-08 Thread Gary Gregory
Good idea to announce! :-) Gary On Jun 8, 2017 11:19 AM, "Benedikt Ritter" wrote: > Hello, > > the Apache Commons CLI component has been moved to git. The new source > repository location is: > > https://git-wip-us.apache.org/repos/asf/commons-cli.git > > The old SVN

[GitHub] commons-text pull request #45: Added IBM Jdk8 build support to travis-ci con...

2017-06-08 Thread ameyjadiye
GitHub user ameyjadiye opened a pull request: https://github.com/apache/commons-text/pull/45 Added IBM Jdk8 build support to travis-ci configuration You can merge this pull request into a Git repository by running: $ git pull https://github.com/ameyjadiye/commons-text

Re: [LANG] Fix date related test failures on IBM JDKs (Was: Re: [CANCEL][VOTE] Release Apache Commons Lang 3.6 based on RC2)

2017-06-08 Thread Amey Jadiye
*Glad to setup this, I will submit PR soon.* *I'm taking commons-text as test mice, once it's successfully running we can replicate same setup to other components.* *Regards,* *Amey* On Thu, Jun 8, 2017 at 11:31 PM, Gary Gregory wrote: > On Thu, Jun 8, 2017 at 8:55

[ANNOUNCE] Apache Commons CLI moved to git

2017-06-08 Thread Benedikt Ritter
Hello, the Apache Commons CLI component has been moved to git. The new source repository location is: https://git-wip-us.apache.org/repos/asf/commons-cli.git The old SVN source tree has been moved to an archive location at: https://svn.apache.org/repos/asf/commons/_moved_to_git/cli Have

Re: [LANG] Fix date related test failures on IBM JDKs (Was: Re: [CANCEL][VOTE] Release Apache Commons Lang 3.6 based on RC2)

2017-06-08 Thread Gary Gregory
On Thu, Jun 8, 2017 at 8:55 AM, Amey Jadiye wrote: > Hi All/Gary/Benedikt/Bruno, > > May be i'm late in this discussion, just had thought around ibmjdk8 that > can we add ibm-jdk8 in our travis.yml file for checking builds against it ? > might be helpful to avoid problem we

[Commons Wiki] Update of "MovingToGit" by BenediktRitter

2017-06-08 Thread Apache Wiki
Dear Wiki user, You have subscribed to a wiki page or wiki category on "Commons Wiki" for change notification. The "MovingToGit" page has been changed by BenediktRitter: https://wiki.apache.org/commons/MovingToGit?action=diff=6=7 Comment: Better formatting == Migrate code to new

Re: [VOTE] Release Apache Commons Lang 3.6 based on RC3

2017-06-08 Thread Rob Tompkins
+1 (non-binding) Signatures ok mvn clean site works in distro. > On Jun 8, 2017, at 1:09 PM, Gary Gregory wrote: > >> On Thu, Jun 8, 2017 at 9:57 AM, sebb wrote: >> >>> On 8 June 2017 at 17:19, Gary Gregory wrote: On

[Commons Wiki] Update of "MovingToGit" by BenediktRitter

2017-06-08 Thread Apache Wiki
Dear Wiki user, You have subscribed to a wiki page or wiki category on "Commons Wiki" for change notification. The "MovingToGit" page has been changed by BenediktRitter: https://wiki.apache.org/commons/MovingToGit?action=diff=5=6 Comment: Describe what to do in order to migrate code from svn

[Commons Wiki] Update of "MovingToGit" by BenediktRitter

2017-06-08 Thread Apache Wiki
Dear Wiki user, You have subscribed to a wiki page or wiki category on "Commons Wiki" for change notification. The "MovingToGit" page has been changed by BenediktRitter: https://wiki.apache.org/commons/MovingToGit?action=diff=4=5 Comment: Document how to request git repositories Notes on

Re: [VOTE] Release Apache Commons Lang 3.6 based on RC3

2017-06-08 Thread Gary Gregory
On Thu, Jun 8, 2017 at 9:57 AM, sebb wrote: > On 8 June 2017 at 17:19, Gary Gregory wrote: > > On Thu, Jun 8, 2017 at 5:38 AM, Simon Spero wrote: > > > >> [A Note, not a vote :) ] > >> > >> 1. Clirr is generally considered

Re: [VOTE] Release Apache Commons Lang 3.6 based on RC3

2017-06-08 Thread Gary Gregory
+1. >From src zip: ASC, MD5, SHA1 OK. RAT and Clirr OK. Clirr does have the warnings discussed on this thread. Running 'mvn clean site' passes with: Apache Maven 3.5.0 (ff8f5e7444045639af65f6095c62210b5713f426; 2017-04-03T12:39:06-07:00) Maven home: C:\Java\apache-maven-3.5.0\bin\.. Java version:

Re: [VOTE] Release Apache Commons Lang 3.6 based on RC3

2017-06-08 Thread sebb
On 8 June 2017 at 17:19, Gary Gregory wrote: > On Thu, Jun 8, 2017 at 5:38 AM, Simon Spero wrote: > >> [A Note, not a vote :) ] >> >> 1. Clirr is generally considered obsolete, as it hadn't been worked on for >> about ten years. japicmp is a good

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

2017-06-08 Thread Rob Tompkins
> On Jun 8, 2017, at 11:02 AM, Matt Sicker wrote: > > Adding the appropriate key to the KEYS file after the fact should still > work. It would have the same cryptographic reliability as being added > beforehand as you can't exactly imitate a key. Yes (mine has been up there

Re: [VOTE] Release Apache Commons Lang 3.6 based on RC3

2017-06-08 Thread Gary Gregory
On Thu, Jun 8, 2017 at 5:38 AM, Simon Spero wrote: > [A Note, not a vote :) ] > > 1. Clirr is generally considered obsolete, as it hadn't been worked on for > about ten years. japicmp is a good replacement, especially for report > generation, and is used in other commons

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

2017-06-08 Thread Gary Gregory
+1 >From src zip: ASC, MD5, SHA1 OK. Build passes with 'mvn clean site'. Tests, CLIRR, RAT and reports OK. Used: Apache Maven 3.5.0 (ff8f5e7444045639af65f6095c62210b5713f426; 2017-04-03T12:39:06-07:00) Maven home: C:\Java\apache-maven-3.5.0\bin\.. Java version: 1.7.0_80, vendor: Oracle

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

2017-06-08 Thread Gary Gregory
On Thu, Jun 8, 2017 at 5:17 AM, Rob Tompkins wrote: > > > > On Jun 8, 2017, at 8:09 AM, sebb wrote: > > > >> On 8 June 2017 at 01:20, Gary Gregory wrote: > >> The ASC does not seem to have a public key.: > >> > >> gpg --verify

Re: [LANG] Fix date related test failures on IBM JDKs (Was: Re: [CANCEL][VOTE] Release Apache Commons Lang 3.6 based on RC2)

2017-06-08 Thread Amey Jadiye
Hi All/Gary/Benedikt/Bruno, May be i'm late in this discussion, just had thought around ibmjdk8 that can we add ibm-jdk8 in our travis.yml file for checking builds against it ? might be helpful to avoid problem we fixed with LANG-1337 I was looking at the way we can do that because the fact that

[GitHub] commons-text issue #44: [TEXT-80]: Fixed confusing StrLookup API

2017-06-08 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-text/pull/44 [![Coverage Status](https://coveralls.io/builds/11887530/badge)](https://coveralls.io/builds/11887530) Coverage remained the same at 96.653% when pulling

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

2017-06-08 Thread Matt Sicker
Adding the appropriate key to the KEYS file after the fact should still work. It would have the same cryptographic reliability as being added beforehand as you can't exactly imitate a key. On 8 June 2017 at 07:17, Rob Tompkins wrote: > > > > On Jun 8, 2017, at 8:09 AM, sebb

[RESULT][VOTE][LAZY] Migrate Apache Commons CLI to git

2017-06-08 Thread Benedikt Ritter
Hi, > Am 05.06.2017 um 13:20 schrieb Benedikt Ritter : > > Hi, > > I’d like to call a vote by lazy consensus for migrating the code base of > Apache Commons CLI to git. > > This vote is successful if nobody raises objections within the next 72 hours, > e.g. until

Re: [VOTE] Release Apache Commons Lang 3.6 based on RC3

2017-06-08 Thread Simon Spero
[A Note, not a vote :) ] 1. Clirr is generally considered obsolete, as it hadn't been worked on for about ten years. japicmp is a good replacement, especially for report generation, and is used in other commons projects. 2. Are the "changes" to the values in CharEncoding really necessary[1]

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

2017-06-08 Thread Rob Tompkins
> On Jun 8, 2017, at 8:09 AM, sebb wrote: > >> On 8 June 2017 at 01:20, Gary Gregory wrote: >> The ASC does not seem to have a public key.: >> >> gpg --verify commons-fileupload-1.3.3-source-release.zip.asc > > That is not the recommended way to

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

2017-06-08 Thread sebb
On 8 June 2017 at 01:20, Gary Gregory wrote: > The ASC does not seem to have a public key.: > > gpg --verify commons-fileupload-1.3.3-source-release.zip.asc That is not the recommended way to check a sig; you also need the target file $ gpg --verify downloaded_file.asc

[GitHub] commons-text issue #44: [TEXT-80]: Fixed confusing StrLookup API

2017-06-08 Thread chtompki
Github user chtompki commented on the issue: https://github.com/apache/commons-text/pull/44 @britter - I would have to test it out, but fair point. It feels like we should always maintain source and binary backwards compatibility for minor version updates. But, if the policy

[GitHub] commons-text issue #44: [TEXT-80]: Fixed confusing StrLookup API

2017-06-08 Thread britter
Github user britter commented on the issue: https://github.com/apache/commons-text/pull/44 @chtompki this indicates a source incompatible release. Binary compatible means that you can swap out the 1.1 jar with the 1.2 jar without a recompile. Would that work? I can't recall

[GitHub] commons-text issue #44: [TEXT-80]: Fixed confusing StrLookup API

2017-06-08 Thread chtompki
Github user chtompki commented on the issue: https://github.com/apache/commons-text/pull/44 @britter if I declare a class ```java package com.rt; import org.apache.commons.text.StrLookup; public class TextTester extends StrLookup { public String

Re: [CANCEL][VOTE] Release Commons Fileupload 1.3.3 based on RC5

2017-06-08 Thread Rob Tompkins
> On Jun 8, 2017, at 5:46 AM, sebb wrote: > > On 8 June 2017 at 01:36, Rob Tompkins > wrote: >> >> >>> On Jun 7, 2017, at 8:31 PM, Gary Gregory wrote: >>> On Wed, Jun 7, 2017 at 5:30 PM, Gary

Re: NUMBERS-40: Review exception usage in package "o.a.c.numbers.gamma"

2017-06-08 Thread Gilles
Hi. On Wed, 7 Jun 2017 17:00:46 -0500, Tharaka De Silva wrote: Hello everyone, I am new to the ASF community Welcome. and decided to grab something easy to attempt. I decided to take a shot at: https://issues.apache.org/jira/browse/NUMBERS-40. Easy to change is not always similar to easy

Re: [VOTE] Release Apache Commons Lang 3.6 based on RC3

2017-06-08 Thread Bruno P. Kinoshita
[ X ] +1 Release these artifacts Build passing for Oracle JDK 7, Oracle JDK 8, and IBM JDK 8. Thanks a lot for preparing the release, Bruno From: Benedikt Ritter To: Commons Developers List Sent: Thursday, 8 June

Re: [CANCEL][VOTE] Release Commons Fileupload 1.3.3 based on RC5

2017-06-08 Thread sebb
On 8 June 2017 at 01:36, Rob Tompkins wrote: > > >> On Jun 7, 2017, at 8:31 PM, Gary Gregory wrote: >> >>> On Wed, Jun 7, 2017 at 5:30 PM, Gary Gregory wrote: >>> >>> >>> On Wed, Jun 7, 2017 at 5:27 PM, Rob Tompkins

[VOTE] Release Apache Commons Lang 3.6 based on RC3

2017-06-08 Thread Benedikt Ritter
Hello, we have fixed quite a few bugs and added some nice new features since Commons Lang 3.5 was released, so I would like to release Commons Lang 3.6 based on RC3. The following issues have been fixed since RC2: - Site build now works from source distribution - IBM JDK test failures have been

[GitHub] commons-text issue #44: [TEXT-80]: Fixed confusing StrLookup API

2017-06-08 Thread britter
Github user britter commented on the issue: https://github.com/apache/commons-text/pull/44 @ameyjadiye the best would be to rebase your branch against master an dann do a force push. You can do it like this (if you have configured this repository as remote with name upstream):

[GitHub] commons-text issue #44: [TEXT-80]: Fixed confusing StrLookup API

2017-06-08 Thread ameyjadiye
Github user ameyjadiye commented on the issue: https://github.com/apache/commons-text/pull/44 Hi @britter, there is no issue even in checkstyle with my changes, above Travis build failed because there was some trailing space issue in master and which you have already fixed. merging

Re: [LANG] Add Automatic-Module-Name MANIFEST entry

2017-06-08 Thread Benedikt Ritter
Hi, > Am 07.06.2017 um 15:09 schrieb Jörg Schaible : > > Stefan Bodewig wrote: > >> On 2017-06-07, Benedikt Ritter wrote: >> >>> here [1] is my proposal on how to add the Automatic-Module-Name entry >>> to MANIFEST. This just duplicates the maven-jar-plugin

[GitHub] commons-text issue #44: [TEXT-80]: Fixed confusing StrLookup API

2017-06-08 Thread britter
Github user britter commented on the issue: https://github.com/apache/commons-text/pull/44 @chtompki did you run chirr manually? Because it was checkstyle which caused the Travis build to fail (trailing white spaces). I think this change should not break BC. @ameyjadiye can you