Re: [vfs] VFS 2.2 and Java 9

2017-10-02 Thread Gary Gregory
OK, I will not spend time on this and create a JIRA so we have it documented. RC2 should be out soon. Gary On Oct 2, 2017 13:03, "Pascal Schumacher" wrote: > Am 02.10.2017 um 19:46 schrieb Gary Gregory: > >> Hi All: >> >> I just canceled 2.2 RC1 due to: >> >> - Some silly test failures on non-

[GitHub] commons-codec pull request #9: CODEC-240 Percent-Encoding codec proposal

2017-10-02 Thread sermojohn
GitHub user sermojohn opened a pull request: https://github.com/apache/commons-codec/pull/9 CODEC-240 Percent-Encoding codec proposal You can merge this pull request into a Git repository by running: $ git pull https://github.com/sermojohn/commons-codec CODEC-240 Alternativel

Re: [vfs] VFS 2.2 and Java 9

2017-10-02 Thread Pascal Schumacher
Am 02.10.2017 um 19:46 schrieb Gary Gregory: Hi All: I just canceled 2.2 RC1 due to: - Some silly test failures on non-Windows platform - Test failures on Java 9 The fixes for the tests are already in trunk. For Java 9, the current behavior is the same as we have in 2.1: the two tests fail in

[vfs] VFS 2.2 and Java 9

2017-10-02 Thread Gary Gregory
Hi All: I just canceled 2.2 RC1 due to: - Some silly test failures on non-Windows platform - Test failures on Java 9 The fixes for the tests are already in trunk. For Java 9, the current behavior is the same as we have in 2.1: the two tests fail in the same way: Tests run: 84, Failures: 0, Err

Re: [VOTE] Release Apache Commons VFS 2.2 from RC1.

2017-10-02 Thread Gary Gregory
FTR, I went back to the 2.1 tag and tested 'mvn clean test' on Windows 10 and Maven 3.5.0 and got: - Oracle Java 1.7.0_80: PASS - Oracle Java 1.8.0_144: PASS - Oracle Java 9 release: FAIL in the same way. Gary On Sun, Oct 1, 2017 at 4:11 AM, Benedikt Ritter wrote: > Hello Gary, > > - signature

[CANCEL][VOTE] Release Apache Commons VFS 2.2 from RC1.

2017-10-02 Thread Gary Gregory
Due to the -1 VOTE from Benedikt, I am canceling this VOTE and will roll out RC2 ASAP. Gary On Sun, Oct 1, 2017 at 8:08 AM, Gary Gregory wrote: > I also had not considered building on Java 9 since Java 9 has made such a > mess with backward compatibility and a lot of Maven plugins fail on Java

Re: [IO] Design of ByteOrderFactory (Was: svn commit: r1301835 - in /commons/proper/io/trunk: RELEASE-NOTES.txt src/changes/ src/changes/changes.xml src/changes/release-notes.vm)

2017-10-02 Thread Gary Gregory
Hi Benedikt, First, thank you for being the RM here. Granted, I might have documented this new class better (and made the ctor private.) That said, while it is a new class for 2.6, we cannot RERO and change the class name later. I do believe that in this case, the narrow class name of "Factory"

[IO] Design of ByteOrderFactory (Was: svn commit: r1301835 - in /commons/proper/io/trunk: RELEASE-NOTES.txt src/changes/ src/changes/changes.xml src/changes/release-notes.vm)

2017-10-02 Thread Benedikt Ritter
Hey Gary, > Am 02.10.2017 um 16:15 schrieb Gary Gregory : > > I am not happy about the change of class name from ByteOrderFactory to > ByteOrderUtils. I am also did a knee jerk -1 when I saw the commit go by > but I have in the middle of moving. > > - A ByteOrderFactory creates ByteOrder objects

Re: svn commit: r1301835 - in /commons/proper/io/trunk: RELEASE-NOTES.txt src/changes/ src/changes/changes.xml src/changes/release-notes.vm

2017-10-02 Thread Gary Gregory
I am not happy about the change of class name from ByteOrderFactory to ByteOrderUtils. I am also did a knee jerk -1 when I saw the commit go by but I have in the middle of moving. - A ByteOrderFactory creates ByteOrder objects and that's it. The intent is clear and focused. - The use of ByteOrder

Re: [lang][text] TEXT-101: RandomStringUtils

2017-10-02 Thread Amey Jadiye
No worries Pascal, it was good experience to change core logic of RandomStringUtils, also I'm free to pickup something else now ;-) Regards, Amey On Mon, Oct 2, 2017, 3:49 PM Pascal Schumacher wrote: > Sorry about wasting your work porting RandomStringUtils to commons-text. :( > > Sorry, > Pasc

[GitHub] commons-text issue #66: Adding validation for pair

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

[GitHub] commons-text pull request #66: Adding validation for pair

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

Re: [lang][text] TEXT-101: RandomStringUtils

2017-10-02 Thread Pascal Schumacher
Sorry about wasting your work porting RandomStringUtils to commons-text. :( Sorry, Pascal Am 29.09.2017 um 17:18 schrieb Amey Jadiye: Okey, no problem folks. I'm going to close my PR. also we might need to release lang-3.6.1 for undoing deprecation of RandomStringUtils if lang-3.7 is far. Reg

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

2017-10-02 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue: https://github.com/apache/commons-collections/pull/27 test failure on java 9: `ListIteratorWrapperTest.testRemove:116 » ServiceConfiguration sun.util.locale` --- --

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

2017-10-02 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-collections/pull/27 [![Coverage Status](https://coveralls.io/builds/13523031/badge)](https://coveralls.io/builds/13523031) Coverage decreased (-0.02%) to 85.118% when pulling **61ed424d6841f4

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

2017-10-02 Thread PascalSchumacher
GitHub user PascalSchumacher opened a pull request: https://github.com/apache/commons-collections/pull/27 travis: add java 9 to build pom.xml: add java 9 profile to make build pass You can merge this pull request into a Git repository by running: $ git pull https://github.com/P

Re: [COLLECTIONS] Test failures on Windows

2017-10-02 Thread Pascal Schumacher
Looks like the random failures are not limited to window. The travis build (ubuntu), just failed with 44 failures, e.g. AbstractMultiValuedMapTest$TestMultiValuedMapAsMap>AbstractMapTest.testMapToString:745->AbstractMapTest.verify:1947->AbstractMapTest.verifyMap:1958 hashCodes should be the sam

[GitHub] commons-text pull request #67: travis: remove travis profile from pom and us...

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

[GitHub] commons-text issue #67: travis: remove travis profile from pom and use travi...

2017-10-02 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue: https://github.com/apache/commons-text/pull/67 The reason for the coveralls failure: ``` Failed to execute goal org.eluder.coveralls:coveralls-maven-plugin:4.3.0:report (default-cli) on project commo

[GitHub] commons-text issue #67: travis: remove travis profile from pom and use travi...

2017-10-02 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-text/pull/67 [![Coverage Status](https://coveralls.io/builds/13522420/badge)](https://coveralls.io/builds/13522420) Coverage remained the same at 98.236% when pulling **778345ee86f0c5e780fd2c