RFR: 8282493: Add --with-jcov-modules convenience option [v5]

Magnus Ihse Bursie ihse at openjdk.org
Wed May 7 11:25:19 UTC 2025


On Wed, 7 May 2025 09:18:29 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

>> Allow the user to set "--with-jcov-modules=java.desktop", instead of "--with-jcov-filters=--include-modules java.desktop", since this is one of the common use cases for the filtering.
>
> Magnus Ihse Bursie has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add missing \

Obviously I did not run the test to it's completion to get a jcov report before; my bad. :-(

However, when I'm now trying to run some more comprehensive testing with jcov, I get multiple test failures. Most are along the lines of:


STDERR:
[Jstatd-Thread] Picked up JAVA_TOOL_OPTIONS: -Xms64m -Xmx4g
[Jstatd-Thread] Picked up _JAVA_OPTIONS: -Xms64m -Xmx4g
[Jstatd-Thread] WARNING: jstatd is deprecated and will be removed in a future release.
java.lang.RuntimeException: Output does not match the pattern
Picked up JAVA_TOOL_OPTIONS: -Xms64m -Xmx4g: expected true, was false
        at jdk.test.lib.Asserts.fail(Asserts.java:715)


JAVA_TOOL_OPTIONS and _JAVA_OPTIONS are setup in lines 127...130 in RunTests.gmk. 

@mrserb  Is this expected behavior, or am I doing something wrong?

-------------

PR Comment: https://git.openjdk.org/jdk/pull/24481#issuecomment-2858200929


More information about the build-dev mailing list