RFR: 8368754: runtime/cds/appcds/SignedJar.java log regex is to strict

David Holmes dholmes at openjdk.org
Mon Sep 29 02:38:12 UTC 2025


On Fri, 26 Sep 2025 09:06:40 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:

> Any logging which gets enabled (via `-verbose:class` which the test uses, or externally from JTREG) and logs before the line this test is checking for with a tag set which is longer than `class,load` makes the test fail.
> Example:
> Test expects: 
>  * `"[class,load] Hello source: [...]"`
>  
> What it gets when logging with `class,unload` prior to this specific log line: 
>  * `"[class,load ] Hello source: [...]"`
> 
> I suggest we simply allow for any whitespace after the log tags and before the `.`. Unclear why we use `.` and not `]`. But think it works just as well.

LGTM2.

FWIW using . instead of [ is rather obscure for the casual reader.

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27517#pullrequestreview-3277694540


More information about the hotspot-runtime-dev mailing list