Re: [COLLECTIONS] Test failures on Windows

2017-10-11 Thread Bruno P. Kinoshita
It was merged in  https://github.com/apache/commons-collections/commit/eed8a7adb3de7441751e846f4a75d110dd205f23. I closed the pull request. Thanks to all that helped testing & troubleshooting this interesting issue! I believe Commons Collections should now be in a state ready for a release. CheersB

[GitHub] commons-collections pull request #28: COLLECTIONS-661: fix for concurrency i...

2017-10-11 Thread kinow
Github user kinow closed the pull request at: https://github.com/apache/commons-collections/pull/28 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.o

[GitHub] commons-collections issue #28: COLLECTIONS-661: fix for concurrency issue in...

2017-10-11 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/commons-collections/pull/28 Merged inhttps://github.com/apache/commons-collections/commit/eed8a7adb3de7441751e846f4a75d110dd205f23. Closing it. --- --

[GitHub] commons-collections pull request #28: COLLECTIONS-661: fix for concurrency i...

2017-10-11 Thread kinow
Github user kinow closed the pull request at: https://github.com/apache/commons-collections/pull/28 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.o

Re: [COLLECTIONS] Test failures on Windows

2017-10-11 Thread Benedikt Ritter
Awesome, let’s merge this! > Am 11.10.2017 um 13:06 schrieb Andreas Kuhtz : > > Hi, > > Tested with the changes of your PR multiple times with Win7-64bit and JDK7 > and JDK8 without any issues. > > Cheers > Andreas > > > > 2017-10-11 10:03 GMT+02:00 Bruno P. Kinoshita < > brunodepau...@yahoo

[GitHub] commons-collections issue #27: travis: add java 9 to build

2017-10-11 Thread vamsi-kavuri
Github user vamsi-kavuri commented on the issue: https://github.com/apache/commons-collections/pull/27 #30 would resolve build issues in java9 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For addi

[GitHub] commons-collections issue #30: COLLECTIONS-662 : Override Jacoco version for...

2017-10-11 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-collections/pull/30 [![Coverage Status](https://coveralls.io/builds/13681730/badge)](https://coveralls.io/builds/13681730) Coverage remained the same at 86.616% when pulling **627b825a24eb03f

[GitHub] commons-collections pull request #30: COLLECTIONS-662 : Override Jacoco vers...

2017-10-11 Thread vamsi-kavuri
GitHub user vamsi-kavuri opened a pull request: https://github.com/apache/commons-collections/pull/30 COLLECTIONS-662 : Override Jacoco version for Java9 compatibility Overriding Jacoco version fixed the failures in Java 9. You can merge this pull request into a Git repository by r

Re: [COLLECTIONS] Test failures on Windows

2017-10-11 Thread Gary Gregory
Works like a charm for me on Windows 10 and Apache Maven 3.5.0 (ff8f5e7444045639af65f6095c62210b5713f426; 2017-04-03T13:39:06-06:00) Maven home: C:\Java\apache-maven-3.5.0\bin\.. Java version: 1.8.0_144, vendor: Oracle Corporation Java home: C:\Program Files\Java\jdk1.8.0_144\jre Default locale: e

[GitHub] commons-collections issue #29: Added unit tests to increase code coverage

2017-10-11 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue: https://github.com/apache/commons-collections/pull/29 Thanks! --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev

[GitHub] commons-collections pull request #29: Added unit tests to increase code cove...

2017-10-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-collections/pull/29 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.

[GitHub] commons-collections pull request #29: Added unit tests to increase code cove...

2017-10-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-collections/pull/29 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.

[GitHub] commons-collections issue #29: Added unit tests to increase code coverage

2017-10-11 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-collections/pull/29 [![Coverage Status](https://coveralls.io/builds/13678056/badge)](https://coveralls.io/builds/13678056) Coverage increased (+1.5%) to 86.616% when pulling **d6e4c2e98b525b4

[GitHub] commons-collections pull request #29: Added unit tests to increase coverage

2017-10-11 Thread vamsi-kavuri
GitHub user vamsi-kavuri opened a pull request: https://github.com/apache/commons-collections/pull/29 Added unit tests to increase coverage You can merge this pull request into a Git repository by running: $ git pull https://github.com/vamsi-kavuri/commons-collections Unit_tes

Re: [RESULT][VOTE] Release Commons Configuration 2.2 based on RC1

2017-10-11 Thread Gary Gregory
Can you release from Nexus ASAP SVP? Gary On Oct 11, 2017 14:24, "Oliver Heger" wrote: > The vote to release Commons Configuration 2.2 based on RC1 passed with > the following binding votes: > > Bruno P. Kinoshita: +1 > Gary Gregory: +1 > Rob Tompkins: +1 > Oliver Heger: +1 >

[RESULT][VOTE] Release Commons Configuration 2.2 based on RC1

2017-10-11 Thread Oliver Heger
The vote to release Commons Configuration 2.2 based on RC1 passed with the following binding votes: Bruno P. Kinoshita: +1 Gary Gregory: +1 Rob Tompkins: +1 Oliver Heger: +1 No other votes were cast. Thanks to all who took the time to review the RC. I will start the release pro

Re: [VOTE] Release Commons Configuration 2.2 based on RC1

2017-10-11 Thread Oliver Heger
My own +1 Oliver Am 08.10.2017 um 22:24 schrieb Oliver Heger: > Hi all, > > there was a request for a new release on the ML, so here we go. > > Configuration 2.2 RC1 is available for review here: > https://dist.apache.org/repos/dist/dev/commons/configuration/ (rev. 22170) > > The tag is here:

[GitHub] commons-text issue #68: TEXT-74 Introduced new StrSubstitutor attribute to d...

2017-10-11 Thread arend-von-reinersdorff
Github user arend-von-reinersdorff commented on the issue: https://github.com/apache/commons-text/pull/68 @sermojohn Great, thanks :-) --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional c

Re: [COLLECTIONS] Test failures on Windows

2017-10-11 Thread Andreas Kuhtz
Hi, Tested with the changes of your PR multiple times with Win7-64bit and JDK7 and JDK8 without any issues. Cheers Andreas 2017-10-11 10:03 GMT+02:00 Bruno P. Kinoshita < brunodepau...@yahoo.com.br.invalid>: > Done. > https://github.com/apache/commons-collections/pull/28 > > Tested on Windows

[GitHub] commons-rng pull request #5: Feature rng 37

2017-10-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-rng/pull/5 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

Re: [COLLECTIONS] Test failures on Windows

2017-10-11 Thread Bruno P. Kinoshita
Done. https://github.com/apache/commons-collections/pull/28 Tested on Windows 10 + JDK8, and Ubuntu LTS + JDK8. Would be good if someone else with a Windows version could try it too, and if others could have a look at the suggested solution in the pull request to see if there's a better way of d

[GitHub] commons-collections issue #28: COLLECTIONS-661: fix for concurrency issue in...

2017-10-11 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-collections/pull/28 [![Coverage Status](https://coveralls.io/builds/13663731/badge)](https://coveralls.io/builds/13663731) Coverage increased (+0.008%) to 85.126% when pulling **6e8951ed0325a

[GitHub] commons-collections pull request #28: COLLECTIONS-661: fix for concurrency i...

2017-10-11 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/commons-collections/pull/28 COLLECTIONS-661: fix for concurrency issue in HashSetValuedHashMapTest The `getMap()` method, when testing a `HashSetValuedHashMap`, would return an object of this type. Which is an instan