[jdk8u-dev] RFR: 8152432: Implement setting jtreg @requires properties vm.flavor, vm.bits, vm.compMode [v2]
Zdenek Zambersky
zzambers at openjdk.org
Thu Mar 16 12:26:50 UTC 2023
> This is backport [1][2] which adds support for jtreg vm properties (`vm.flavor`, `vm.compMode`, `vm.bits`) for use in `@requires`
>
> First commit is backport Itself.
>
> Second commit switches some tests back to use `@requires` (2 are basically reverts [3][4] of jdk8u specific changes, 3rd updated test is not present in newer jdks [5][6])
> These updated tests were used to test new properties, so I included them here. (However I can exclude them if requested)
>
> Testing:
> Updated tests are part of hotspot tier1, which is executed in GitHub (Which confirms it works).
>
> [1] https://github.com/openjdk/jdk9u/commit/24a9e0ac188a37dc57cc4d1bb8d8635abb4c4f89
> [2] https://github.com/openjdk/jdk9u/commit/2b7a70a9d3d9b4ca94605d22d3d4f3c78a5c8d8b
> [3] https://github.com/openjdk/jdk8u-dev/pull/222
> [4] https://github.com/openjdk/jdk8u-dev/pull/182
> [5] https://bugs.openjdk.org/browse/JDK-8154204
> [6] https://github.com/openjdk/jdk8u-dev/pull/181
Zdenek Zambersky has updated the pull request incrementally with one additional commit since the last revision:
DefaultUseWithClient.java: remove redundant comment line
-------------
Changes:
- all: https://git.openjdk.org/jdk8u-dev/pull/286/files
- new: https://git.openjdk.org/jdk8u-dev/pull/286/files/6ec2df7e..daa900e5
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=286&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=286&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod
Patch: https://git.openjdk.org/jdk8u-dev/pull/286.diff
Fetch: git fetch https://git.openjdk.org/jdk8u-dev pull/286/head:pull/286
PR: https://git.openjdk.org/jdk8u-dev/pull/286
More information about the jdk8u-dev
mailing list