RFR: 8252003: remove usage of PropertyResolvingWrapper in vmTestbase/nsk/jvmti [v2]
Serguei Spitsyn
sspitsyn at openjdk.java.net
Wed Sep 30 21:07:43 UTC 2020
On Wed, 30 Sep 2020 20:56:40 GMT, Igor Ignatyev <iignatyev at 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
More information about the serviceability-dev
mailing list