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