On Thu, 19 Nov 2020 16:34:36 GMT, Marius Volkhart <github.com+1709517+mariusvolkh...@openjdk.org> wrote:
>> @MariusVolkhart I'll work with you getting the fix in and get the test >> running using jtreg. Give me a few moments to reproduce. > > Thanks @jerboaa > > I have a JBS number now: > [8256515](https://bugs.java.com/bugdatabase/view_bug.do?bug_id=JDK-8256515). > Since opening the PR I've learned that I've gone about the process backwards, > and should have found a Sponsor before making the PR. As such, I haven't > updated the commit message to avoid sending a premature RFR to the mailing > list @MariusVolkhart I think doing a PR is fine before you find a sponsor provided you're fairly certain it's an actual bug. It'll require one to then send an email to the mailing lists asking for a sponsor. That email could potentially reference a PR. That should be fine. The sponsor (committer) will then help you get it integrated. It somewhat depends on the issue. No hard-and-fast rules that I'm aware of in that regard. @MariusVolkhart Here is a PR for your branch so the test works with jtreg: https://github.com/MariusVolkhart/jdk/pull/1 Fails before the JDK patch and passes after. Run it with: $ rm -rf JTwork JTreport && jtreg -jdk:./build/linux-x86_64-server-release/images/jdk -verbose:summary test/jdk/javax/xml/jaxp/8256515/XmlInputFactoryTest.java Passed: javax/xml/jaxp/8256515/XmlInputFactoryTest.java Test results: passed: 1 Results are in *.jtr files: $ find JTwork/ -name *.jtr JTwork/javax/xml/jaxp/8256515/XmlInputFactoryTest.jtr Or using the test framework of OpenJDK with: $ bash configure --with-jtreg=/path/to/jtreg [...] $ make test TEST="jdk/javax/xml/jaxp/8256515/XmlInputFactoryTest.java" HTH ------------- PR: https://git.openjdk.java.net/jdk/pull/1056