RFR: 7904084: Unset javaOpts and jtregOpts in jtreg.sh [v2]
SendaoYan
syan at openjdk.org
Thu Nov 20 08:44:56 UTC 2025
> Hi all,
>
> In jtreg.sh it will combine all the java options and jtreg options in the `for i in "$@"` loop. SHELL use javaOpts and jtregOpts as global varable by default, this means jtreg.sh will receive the value of javaOpts and jtregOpts outside. I think this unexpected.
> To avoid this siutation, I think we should unset javaOpts and jtregOpts before use it.
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
-------------
Changes:
- all: https://git.openjdk.org/jtreg/pull/291/files
- new: https://git.openjdk.org/jtreg/pull/291/files/bb833f9b..5e2b37ae
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jtreg&pr=291&range=01
- incr: https://webrevs.openjdk.org/?repo=jtreg&pr=291&range=00-01
Stats: 161 lines in 12 files changed: 86 ins; 32 del; 43 mod
Patch: https://git.openjdk.org/jtreg/pull/291.diff
Fetch: git fetch https://git.openjdk.org/jtreg.git pull/291/head:pull/291
PR: https://git.openjdk.org/jtreg/pull/291
More information about the jtreg-dev
mailing list