I like that. I’ll get that sorted out. -Rob
> On Apr 10, 2019, at 10:28 AM, Gary Gregory <garydgreg...@gmail.com> wrote: > > I think UnsupportedOperationException with a Java 12 comment, but we should > also add @Deprecated with a comment about it not working at all on Java 12. > > Gary > > On Wed, Apr 10, 2019 at 8:55 AM Rob Tompkins <chtom...@gmail.com> wrote: > >> Looks like we get an exception because the final modifier isn’t removable >> in java 12+ and for good reason [1]. So, we probably should throw an >> exception, but both of the exceptions that arise out of the jvm here >> https://github.com/apache/commons-lang/blob/master/src/main/java/org/apache/commons/lang3/reflect/FieldUtils.java#L736 >> < >> https://github.com/apache/commons-lang/blob/master/src/main/java/org/apache/commons/lang3/reflect/FieldUtils.java#L736> >> are checked exceptions. Specifically we get a “NoSuchFieldException” when >> we attempt to retrieve “modifiers” from the “Field.” For the sake of >> retaining BC, I would think that we would in turn convert that Exception to >> an “UnsupportedOperationException” or a “TypeNotPresentException” (maybe >> ??) for the sake of getting to an unchecked exception. >> >> Open to thoughts here, >> >> -Rob >> >> >> [1] >> http://mail.openjdk.java.net/pipermail/core-libs-dev/2018-November/056486.html >> >> >>> On Apr 10, 2019, at 5:09 AM, Benedikt Ritter <brit...@apache.org> wrote: >>> >>> Damn it, I had it on my list to fix those. Sorry, Rob! >>> >>> Am Mi., 10. Apr. 2019 um 03:33 Uhr schrieb Rob Tompkins < >> chtom...@gmail.com >>>> : >>> >>>> >>>> >>>>> On Apr 9, 2019, at 9:15 PM, Gary Gregory <garydgreg...@gmail.com> >> wrote: >>>>> >>>>> +0... see Java 12 testing below. >>>> >>>> That +0 is almost worth cancelling for in my opinion. I’ll wait until >>>> tomorrow to see if anyone else wants to have an opinion here, and then >> I’ll >>>> have another go at it. >>>> >>>> -Rob >>>> >>>>> >>>>> Nit in the release notes: Replace "Java 8.0" with Java 8". >>>>> >>>>> From the git tag, ran 'mvn -V clean package' OK for: >>>>> >>>>> Apache Maven 3.6.0 (97c98ec64a1fdfee7767ce5ffb20918da4f719f3; >>>>> 2018-10-24T14:41:47-04:00) >>>>> Maven home: C:\Java\apache-maven-3.6.0\bin\.. >>>>> Java version: 1.8.0_202, vendor: Oracle Corporation, runtime: >> C:\Program >>>>> Files\Java\jdk1.8.0_202\jre >>>>> Default locale: en_US, platform encoding: Cp1252 >>>>> OS name: "windows 10", version: "10.0", arch: "amd64", family: >> "windows" >>>>> >>>>> Apache Maven 3.6.0 (97c98ec64a1fdfee7767ce5ffb20918da4f719f3; >>>>> 2018-10-24T14:41:47-04:00) >>>>> Maven home: C:\Java\apache-maven-3.6.0\bin\.. >>>>> Java version: 11.0.2, vendor: Oracle Corporation, runtime: C:\Program >>>>> Files\Java\openjdk\jdk-11.0.2 >>>>> Default locale: en_US, platform encoding: Cp1252 >>>>> OS name: "windows 10", version: "10.0", arch: "amd64", family: >> "windows" >>>>> >>>>> BUT on Java 12: >>>>> >>>>> Apache Maven 3.6.0 (97c98ec64a1fdfee7767ce5ffb20918da4f719f3; >>>>> 2018-10-24T14:41:47-04:00) >>>>> Maven home: C:\Java\apache-maven-3.6.0\bin\.. >>>>> Java version: 12, vendor: Oracle Corporation, runtime: C:\Program >>>>> Files\Java\openjdk\jdk-12 >>>>> Default locale: en_US, platform encoding: Cp1252 >>>>> OS name: "windows 10", version: "10.0", arch: "amd64", family: >> "windows" >>>>> >>>>> I get: >>>>> >>>>> [ERROR] Failures: >>>>> [ERROR] FieldUtilsTest.testRemoveFinalModifier:998 expected: <false> >>>> but >>>>> was: <true> >>>>> [ERROR] FieldUtilsTest.testRemoveFinalModifierWithAccess:1009 >> expected: >>>>> <false> but was: <true> >>>>> [INFO] >>>>> [ERROR] Tests run: 5774, Failures: 2, Errors: 0, Skipped: 4 >>>>> >>>>> On Java 13-EA: >>>>> >>>>> Apache Maven 3.6.0 (97c98ec64a1fdfee7767ce5ffb20918da4f719f3; >>>>> 2018-10-24T14:41:47-04:00) >>>>> Maven home: C:\Java\apache-maven-3.6.0\bin\.. >>>>> Java version: 13-ea, vendor: Oracle Corporation, runtime: C:\Program >>>>> Files\Java\openjdk\jdk-13 >>>>> Default locale: en_US, platform encoding: Cp1252 >>>>> OS name: "windows 10", version: "10.0", arch: "amd64", family: >> "windows" >>>>> >>>>> I get: >>>>> >>>>> [INFO] ------------------------------------------------------- >>>>> [INFO] T E S T S >>>>> [INFO] ------------------------------------------------------- >>>>> [WARNING] Corrupted STDOUT by directly writing to native stream in >> forked >>>>> JVM 1. See FAQ web page and the dump file >>>>> >>>> >> C:\temp\rc\commons-lang\target\surefire-reports\2019-04-09T21-13-37_709-jvmRun1.dumpstream >>>>> [INFO] >>>>> [INFO] Results: >>>>> [INFO] >>>>> [INFO] Tests run: 0, Failures: 0, Errors: 0, Skipped: 0 >>>>> [INFO] >>>>> [INFO] >>>>> >> ------------------------------------------------------------------------ >>>>> [INFO] BUILD FAILURE >>>>> [INFO] >>>>> >> ------------------------------------------------------------------------ >>>>> [INFO] Total time: 27.315 s >>>>> [INFO] Finished at: 2019-04-09T21:13:38-04:00 >>>>> [INFO] >>>>> >> ------------------------------------------------------------------------ >>>>> [ERROR] Failed to execute goal >>>>> org.apache.maven.plugins:maven-surefire-plugin:2.22.1:test >> (default-test) >>>>> on project commons-lang3: There are test failures. >>>>> [ERROR] >>>>> [ERROR] Please refer to C:\temp\rc\commons-lang\target\surefire-reports >>>> for >>>>> the individual test results. >>>>> [ERROR] Please refer to dump files (if any exist) [date].dump, >>>>> [date]-jvmRun[N].dump and [date].dumpstream. >>>>> [ERROR] The forked VM terminated without properly saying goodbye. VM >>>> crash >>>>> or System.exit called? >>>>> [ERROR] Command was cmd.exe /X /C ""C:\Program >>>>> Files\Java\openjdk\jdk-13\bin\java" >>>>> >>>> >> -javaagent:C:\\Users\\ggregory\\.m2\\repository\\org\\jacoco\\org.jacoco.agent\\0.8.2\\org.jacoco.agent-0.8.2-runtime.jar=destfile=C:\\temp\\rc\\commons-lang\\target\\jacoco.exec >>>>> -Xmx512m --add-opens java.base/java.lang.reflect=ALL-UNNAMED >> --add-opens >>>>> java.base/java.lang=ALL-UNNAMED -jar >>>>> >>>> >> C:\Users\ggregory\AppData\Local\Temp\surefire5519858828807362694\surefirebooter988663828837655366.jar >>>>> C:\Users\ggregory\AppData\Local\Temp\surefire5519858828807362694 >>>>> 2019-04-09T21-13-37_709-jvmRun1 surefire11717302241379819897tmp >>>>> surefire_07095889889312770815tmp" >>>>> [ERROR] Error occurred in starting fork, check output in log >>>>> [ERROR] Process Exit Code: 1 >>>>> [ERROR] org.apache.maven.surefire.booter.SurefireBooterForkException: >> The >>>>> forked VM terminated without properly saying goodbye. VM crash or >>>>> System.exit called? >>>>> [ERROR] Command was cmd.exe /X /C ""C:\Program >>>>> Files\Java\openjdk\jdk-13\bin\java" >>>>> >>>> >> -javaagent:C:\\Users\\ggregory\\.m2\\repository\\org\\jacoco\\org.jacoco.agent\\0.8.2\\org.jacoco.agent-0.8.2-runtime.jar=destfile=C:\\temp\\rc\\commons-lang\\target\\jacoco.exec >>>>> -Xmx512m --add-opens java.base/java.lang.reflect=ALL-UNNAMED >> --add-opens >>>>> java.base/java.lang=ALL-UNNAMED -jar >>>>> >>>> >> C:\Users\ggregory\AppData\Local\Temp\surefire5519858828807362694\surefirebooter988663828837655366.jar >>>>> C:\Users\ggregory\AppData\Local\Temp\surefire5519858828807362694 >>>>> 2019-04-09T21-13-37_709-jvmRun1 surefire11717302241379819897tmp >>>>> surefire_07095889889312770815tmp" >>>>> [ERROR] Error occurred in starting fork, check output in log >>>>> [ERROR] Process Exit Code: 1 >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.plugin.surefire.booterclient.ForkStarter.fork(ForkStarter.java:669) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.plugin.surefire.booterclient.ForkStarter.run(ForkStarter.java:282) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.plugin.surefire.booterclient.ForkStarter.run(ForkStarter.java:245) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.plugin.surefire.AbstractSurefireMojo.executeProvider(AbstractSurefireMojo.java:1183) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.plugin.surefire.AbstractSurefireMojo.executeAfterPreconditionsChecked(AbstractSurefireMojo.java:1011) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.plugin.surefire.AbstractSurefireMojo.execute(AbstractSurefireMojo.java:857) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:137) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:210) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:156) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:148) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:117) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:81) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build(SingleThreadedBuilder.java:56) >>>>> [ERROR] at >>>>> >>>> >> org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:128) >>>>> [ERROR] at >>>>> org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:305) >>>>> [ERROR] at >>>>> org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:192) >>>>> [ERROR] at >>>>> org.apache.maven.DefaultMaven.execute(DefaultMaven.java:105) >>>>> [ERROR] at >>>> org.apache.maven.cli.MavenCli.execute(MavenCli.java:956) >>>>> [ERROR] at >>>> org.apache.maven.cli.MavenCli.doMain(MavenCli.java:288) >>>>> [ERROR] at >> org.apache.maven.cli.MavenCli.main(MavenCli.java:192) >>>>> [ERROR] at >>>>> java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native >>>>> Method) >>>>> [ERROR] at >>>>> >>>> >> java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) >>>>> [ERROR] at >>>>> >>>> >> java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) >>>>> [ERROR] at >>>>> java.base/java.lang.reflect.Method.invoke(Method.java:567) >>>>> [ERROR] at >>>>> >>>> >> org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:289) >>>>> [ERROR] at >>>>> >>>> >> org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:229) >>>>> [ERROR] at >>>>> >>>> >> org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:415) >>>>> [ERROR] at >>>>> >> org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:356) >>>>> [ERROR] >>>>> [ERROR] -> [Help 1] >>>>> >>>>> Gary >>>>> >>>>> On Tue, Apr 9, 2019 at 3:37 PM Rob Tompkins <chtom...@apache.org> >> wrote: >>>>> >>>>>> We have fixed quite a few bugs and added some significant enhancements >>>>>> since Apache Commons Lang 3.8.1 was released, so I would like to >> release >>>>>> Apache Commons Lang 3.9. >>>>>> >>>>>> Apache Commons Lang 3.9 RC1 is available for review here: >>>>>> https://dist.apache.org/repos/dist/dev/commons/lang/3.9-RC1 (svn >>>>>> revision 33538) >>>>>> >>>>>> The Git tag commons-lang-3.9-RC1 commit for this RC is >>>>>> 8b378885152075b44c5710d6dbcb8df4a5fb3223 which you can browse here: >>>>>> >>>>>> >>>> >> https://gitbox.apache.org/repos/asf?p=commons-lang.git;a=commit;h=8b378885152075b44c5710d6dbcb8df4a5fb3223 >>>>>> You may checkout this tag using: >>>>>> git clone https://gitbox.apache.org/repos/asf/commons-lang.git >>>>>> --branch commons-lang-3.9-RC1 >>>>>> >>>>>> Maven artifacts are here: >>>>>> >>>>>> >>>> >> https://repository.apache.org/content/repositories/orgapachecommons-1433/org/apache/commons/commons-lang3/3.9/ >>>>>> >>>>>> These are the Maven artifacts and their hashes in Nexus: >>>>>> >>>>>> #Nexus SHA-1s >>>>>> commons-lang3-3.9-sources.jar=6dce2c1ea3cbb6c438b2313786a501b38e5ad7b0 >>>>>> commons-lang3-3.9.pom=a0204d47009f50de96af0b87acfd1c6b3b16419a >>>>>> commons-lang3-3.9-javadoc.jar=c643c72d73f70ed7d71783f3ce8f37b057e33777 >>>>>> commons-lang3-3.9.jar=d76bbd8a0213744a14af3ba80ab3ab69b02bdbae >>>>>> commons-lang3-3.9-tests.jar=411ede8701d31196bc28c2c60607e6de47da4922 >>>>>> >>>> >> commons-lang3-3.9-test-sources.jar=a28a97bb8d4fa3e05230d3eed5b8f934f3f55889 >>>>>> >>>>>> >>>>>> #Release SHA-512s >>>>>> #Tue Apr 09 14:23:45 EDT 2019 >>>>>> >>>>>> >>>> >> commons-lang3-3.9-src.tar.gz=8e85013ae7629b205ed39c54f9c14d8d7801c47226373ccb0482a0149dbf9c7654d6b5c81815b594f45b4426917bbe1618db7ba6758665e39771efbb98c49e20 >>>>>> >>>>>> >>>> >> commons-lang3-3.9-bin.tar.gz=8ebaa30565d80d32dbb1ba19047f61027fd043294b85aa1f6f1613c7ac4739da030ff2a5ea2ad119283d13fb5db7b41c1bd784f95828cb9727a3f7e9f302d9fc >>>>>> >>>>>> >>>> >> commons-lang3-3.9-src.zip=30595b6f5df73aca6542bb70c626234d6ec43e2f11316f7a5123982ba100027437b7ecc44bbfd0f665152c3e98e997bc145d7976b33a48526aa6d929a8c1f7ad >>>>>> >>>>>> >>>> >> commons-lang3-3.9-bin.zip=072c67c4793e7f4fa070202ab5ebe3fa29d81bfd23982a00bfd1d752e54991b696042ba791706458f9993d3721198f08fce70a6f9d1f16fd94d6cc54856393f8 >>>>>> >>>>>> >>>>>> I have tested this with 'mvn clean test package site' using: >>>>>> Apache Maven 3.6.0 (97c98ec64a1fdfee7767ce5ffb20918da4f719f3; >>>>>> 2018-10-24T14:41:47-04:00) >>>>>> Maven home: /usr/local/Cellar/maven/3.6.0/libexec >>>>>> Java version: 1.8.0_202, vendor: Amazon.com Inc., runtime: >>>>>> >>>> >> /Library/Java/JavaVirtualMachines/amazon-corretto-8.jdk/Contents/Home/jre >>>>>> Default locale: en_US, platform encoding: UTF-8 >>>>>> OS name: "mac os x", version: "10.14.4", arch: "x86_64", family: "mac" >>>>>> >>>>>> >>>>>> Details of changes since 3.8.1 are in the release notes: >>>>>> >>>>>> >>>> >> https://dist.apache.org/repos/dist/dev/commons/lang/3.9-RC1/RELEASE-NOTES.txt >>>>>> >>>>>> >>>> >> https://dist.apache.org/repos/dist/dev/commons/lang/3.9-RC1/site/changes-report.html >>>>>> >>>>>> Site: >>>>>> https://dist.apache.org/repos/dist/dev/commons/lang/3.9-RC1/site >>>>>> (note some *relative* links are broken and the 3.9 directories are >>>> not >>>>>> yet created - these will be OK once the site is deployed.) >>>>>> >>>>>> CLIRR Report (compared to 3.8.1): >>>>>> >>>>>> >>>> >> https://dist.apache.org/repos/dist/dev/commons/lang/3.9-RC1/site/clirr-report.html >>>>>> >>>>>> JApiCmp Report (compared to 3.8.1): >>>>>> >>>>>> >>>> >> https://dist.apache.org/repos/dist/dev/commons/lang/3.9-RC1/site/japicmp.html >>>>>> >>>>>> RAT Report: >>>>>> >>>>>> >>>> >> https://dist.apache.org/repos/dist/dev/commons/lang/3.9-RC1/site/rat-report.html >>>>>> >>>>>> KEYS: >>>>>> https://www.apache.org/dist/commons/KEYS >>>>>> >>>>>> Please review the release candidate and vote. >>>>>> This vote will close no sooner that 72 hours from now. >>>>>> >>>>>> [ ] +1 Release these artifacts >>>>>> [ ] +0 OK, but... >>>>>> [ ] -0 OK, but really should fix... >>>>>> [ ] -1 I oppose this release because... >>>>>> >>>>>> Thank you, >>>>>> >>>>>> Rob Tompkins, >>>>>> Release Manager (using key B6E73D84EA4FCC47166087253FAAD2CD5ECBB314) >>>>>> >>>>>> For following is intended as a helper and refresher for reviewers. >>>>>> >>>>>> Validating a release candidate >>>>>> ============================== >>>>>> >>>>>> These guidelines are NOT complete. >>>>>> >>>>>> Requirements: Git, Java, Maven. >>>>>> >>>>>> You can validate a release from a release candidate (RC) tag as >> follows. >>>>>> >>>>>> 1) Clone and checkout the RC: >>>>>> >>>>>> git clone https://gitbox.apache.org/repos/asf/lang.git -b >>>>>> commons-lang-3.9-RC1 >>>>>> cd commons-lang-3.9-RC1 >>>>>> >>>>>> 2) Check Apache licenses: >>>>>> >>>>>> mvn apache-rat:check >>>>>> >>>>>> 3) Build the package: >>>>>> >>>>>> mvn -V clean package >>>>>> >>>>>> You can record the Maven and Java version produced by -V in your VOTE >>>>>> reply. >>>>>> >>>>>> 4) Build the site for a single module project: >>>>>> >>>>>> mvn site >>>>>> Check the site reports in: >>>>>> target\site\index.html >>>>>> >>>>>> 4) Build the site for a multi-module project: >>>>>> >>>>>> mvn site >>>>>> mvn site:stage >>>>>> Check the site reports in: >>>>>> target\site\index.html >>>>>> >>>>>> >>>>>> --------------------------------------------------------------------- >>>>>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org >>>>>> For additional commands, e-mail: dev-h...@commons.apache.org >>>>>> >>>>>> >>>> >>>> >>>> --------------------------------------------------------------------- >>>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org >>>> For additional commands, e-mail: dev-h...@commons.apache.org >>>> >>>> >> >> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org