RFR: 7904084: Unset javaOpts and jtregOpts in jtreg.sh [v4]
SendaoYan
syan at openjdk.org
Wed Jan 28 07:07:23 UTC 2026
> 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 four additional commits since the last revision:
- Merge branch 'openjdk:master' into jbs7904084
- Merge branch 'openjdk:master' into jbs7904084
- 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/fb3a5dd1..a3076f53
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jtreg&pr=291&range=03
- incr: https://webrevs.openjdk.org/?repo=jtreg&pr=291&range=02-03
Stats: 196 lines in 18 files changed: 162 ins; 5 del; 29 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