Thanks Jim for pushing the reverting changeset.
PS. Sorry that I broke the build. I did run jrtfs, javac and javap
tests. Didn't realise that faulty jrt: URIs are pervasive in tests (per
JEP jrt: URIs should not have '/modules/' part - which is the bug fix is
about)
-Sundar
On 29/05/19, 1:19 AM, Jim Laskey wrote:
Thank you.
On May 28, 2019, at 4:44 PM, Joe Darcy<joe.da...@oracle.com> wrote:
If this is a straight backport of the previous change, I approve the revision.
Cheers,
-Joe
On 5/28/2019 12:39 PM, Jim Laskey wrote:
Please review. Need to revert a change Sundar did earlier today that is causing
33 tier2 failures.
Cheers,
-- Jim
Reverse webrev:
http://cr.openjdk.java.net/~jlaskey/8224908/webrev/index.html<http://cr.openjdk.java.net/~jlaskey/8224908/webrev/index.html>
JBS:
https://bugs.openjdk.java.net/browse/JDK-8224908<https://bugs.openjdk.java.net/browse/JDK-8224908>
Job:
mach5-one-jdk-13-1151-tier2-20190528-1515-2755946<http://java.se.oracle.com:10065/mdash/jobs/mach5-one-jdk-13-1151-tier2-20190528-1515-2755946>
Original changset:
http://hg.openjdk.java.net/jdk/jdk/rev/63ab89cc3e69<http://hg.openjdk.java.net/jdk/jdk/rev/63ab89cc3e69>