Integrated: 7903780: NPE in new code for LIBRARY.properties

Jonathan Gibbons jjg at openjdk.org
Tue Jul 23 20:52:45 UTC 2024


On Tue, 23 Jul 2024 19:01:05 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:

> As well as the NPE, the following additional minor issues were fixed:
> 
> `--source` is a recent addition for `javac` options; older versions of `javac` still require `-source N`
> 
> `--enable-preview` is required at runtime if any library uses preview features.
> For now, I just moved `usesLibraryCompiledWithPreviewEnabled` from `CompileAction` up into `Action`, so that it can be used from `MainAction`. We might want to consider merging the merging into `RegressionScript.enablePreview`, but that can be done when we clean up the use of `LIBRARY.properties`, caching the results in `LibLocn`.
> 
> We should also investigate the difference between `libLocn == null` and `libLocn.isTest()` 
> 
> After this, all self-tests pass, including the recently new tests for `LIBRARY.properties`.

This pull request has now been integrated.

Changeset: 38d49d16
Author:    Jonathan Gibbons <jjg at openjdk.org>
URL:       https://git.openjdk.org/jtreg/commit/38d49d16657e9d5051ad93426592ddd7bf8f2d98
Stats:     23 lines in 3 files changed: 12 ins; 8 del; 3 mod

7903780: NPE in new code for LIBRARY.properties

Reviewed-by: iris

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

PR: https://git.openjdk.org/jtreg/pull/213


More information about the jtreg-dev mailing list