On Wed, 30 Sep 2020 20:56:40 GMT, Igor Ignatyev <iignat...@openjdk.org> wrote:
>> Hi all, >> >> could you please review the patch which removes `PropertyResolvingWrapper` >> from `vmTestbase/nsk/jvmti` tests? as >> `jtreg` doesn't support spaces in the arguments and doesn't handle `"` in >> any special ways, the patch also: >> - `s/"-javaOpts=/-javaOpts="/` >> - makes `nsk.jvmti.scenarios.general_functions.GF08` to use 2nd arg as >> `verboseType` and 3rd and the rest args >> concatenated as `phrase` and updates the tests accordingly >> - removes spaces and surrounding `"` from `nsk.jvmti.test.property*` >> - removes `"` surrounding `-agentlib:`, replaces spaces in `-agentlib` with >> `,` and updates `ArgumentHandler` to treat >> `,` (as well as ` ` and `~`) as options delimiters, so it's consistent w/ >> `jvmti_tools.cpp` >> >> testing: ✅ `vmTestbase/nsk/jvmti` on {linux,windows,macos}-x64 > > Igor Ignatyev has updated the pull request incrementally with one additional > commit since the last revision: > > Revert "update ArgumentHandler to use the same delimeters as in > jvmti_tools.cpp" Marked as reviewed by sspitsyn (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/370