jdk17u-dev icon indicating copy to clipboard operation
jdk17u-dev copied to clipboard

8272317: jstatd has dependency on Security Manager which needs to be removed

Open shipilev opened this issue 1 year ago • 6 comments
trafficstars

Semi-clean backport to resolve the jstatd dependency on SM. Current jstatd tests are also failing because some recently backported tests do not expect SM deprecation messages, see JDK-8333698.

This also reverts 17u-specific test amendment, JDK-8272317.

Additional testing:

  • [x] MacOS AArch64 server fastdebug, sun/tools/jstatd/ now pass

Progress

  • [x] Change must be properly reviewed (1 review required, with at least 1 Reviewer)
  • [x] Change must not contain extraneous whitespace
  • [x] Commit message must refer to an issue
  • [ ] JDK-8272317 needs maintainer approval
  • [ ] Change requires CSR request JDK-8333700 to be approved

Issues

  • JDK-8272317: jstatd has dependency on Security Manager which needs to be removed (Enhancement - P3)
  • JDK-8333700: jstatd has dependency on Security Manager which needs to be removed (CSR)

Reviewers

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk17u-dev.git pull/2542/head:pull/2542
$ git checkout pull/2542

Update a local copy of the PR:
$ git checkout pull/2542
$ git pull https://git.openjdk.org/jdk17u-dev.git pull/2542/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 2542

View PR using the GUI difftool:
$ git pr show -t 2542

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk17u-dev/pull/2542.diff

Webrev

Link to Webrev Comment

shipilev avatar Jun 06 '24 09:06 shipilev