RFR: 7904084: Unset javaOpts and jtregOpts in jtreg.sh [v2]
SendaoYan
syan at openjdk.org
Thu Jan 29 02:42:15 UTC 2026
On Tue, 2 Dec 2025 12:01:02 GMT, Christian Stein <cstein at openjdk.org> wrote:
>> SendaoYan has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:
>>
>> - Merge branch 'openjdk:master' into jbs7904084
>> - 7904084: Unset javaOpts and jtregOpts in jtreg.sh
>
> Did you see an error when used in combination with another or the same script? If yes, which one(s)?
>
> There's related prior-art in the script for the `DUALCASE` variable. Which is declard and initialized before each use, and `unset` after each use. Would it be possible to the same for `javaOpts` and `jtregOpts`? Or at least `unset` both after use.
>
> Another solution would be to introduce script-`local` variables, right?
Thanks for the reviews @sormuras
Does this PR need second reviewer, or may I integrate this PR now.
-------------
PR Comment: https://git.openjdk.org/jtreg/pull/291#issuecomment-3815092641
More information about the jtreg-dev
mailing list