[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-2d-dev
Subject:    [OpenJDK 2D-Dev] Integrated: 8266459: Implement JEP 411: Deprecate the Security Manager for Removal
From:       Weijun Wang <weijun () openjdk ! java ! net>
Date:       2021-06-02 12:01:33
Message-ID: PsbFcEhvCahb4zpsvIhOlxvAbJK7DTE3-umT3Nsn5dY=.fe038b5f-dc17-4d4f-a19b-ed8c238c7d58 () github ! com
[Download RAW message or body]

On Mon, 17 May 2021 18:23:41 GMT, Weijun Wang <weijun@openjdk.org> wrote:

> Please review this implementation of [JEP 411](https://openjdk.java.net/jeps/411).
> 
> The code change is divided into 3 commits. Please review them one by one.
> 
> 1. https://github.com/openjdk/jdk/commit/576161d15423f58281e384174d28c9f9be7941a1 \
> The essential change for this JEP, including the `@Deprecate` annotations and spec \
> change. It also update the default value of the `java.security.manager` system \
> property to "disallow", and necessary test change following this update. 2. \
> https://github.com/openjdk/jdk/commit/26a54a835e9f84aa528740a7c5c35d07355a8a66 \
> Manual changes to several files so that the next commit can be generated \
> programatically. 3. \
> https://github.com/openjdk/jdk/commit/eb6c566ff9207974a03a53335e0e697cffcf0950 \
> Automatic changes to other source files to avoid javac warnings on deprecation for \
> removal 
> The 1st and 2nd commits should be reviewed carefully. The 3rd one is generated \
> programmatically, see the comment below for more details. If you are only \
> interested in a portion of the 3rd commit and would like to review it as a separate \
> file, please comment here and I'll generate an individual webrev. 
> Due to the size of this PR, no attempt is made to update copyright years for any \
> file to minimize unnecessary merge conflict. 
> Furthermore, since the default value of `java.security.manager` system property is \
> now "disallow", most of the tests calling `System.setSecurityManager()` need to \
> launched with `-Djava.security.manager=allow`. This is covered in a different PR at \
> https://github.com/openjdk/jdk/pull/4071. 
> Update: the deprecation annotations and javadoc tags, build, compiler, core-libs, \
> hotspot, i18n, jmx, net, nio, security, and serviceability are reviewed. Rest are \
> 2d, awt, beans, sound, and swing.

This pull request has now been integrated.

Changeset: 6765f902
Author:    Weijun Wang <weijun@openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/6765f902505fbdd02f25b599f942437cd805cad1
                
Stats:     2755 lines in 826 files changed: 1997 ins; 20 del; 738 mod

8266459: Implement JEP 411: Deprecate the Security Manager for Removal

Co-authored-by: Sean Mullan <mullan@openjdk.org>
Co-authored-by: Lance Andersen <lancea@openjdk.org>
Co-authored-by: Weijun Wang <weijun@openjdk.org>
Reviewed-by: erikj, darcy, chegar, naoto, joehw, alanb, mchung, kcr, prr, lancea

-------------

PR: https://git.openjdk.java.net/jdk/pull/4073


[prev in list] [next in list] [prev in thread] [next in thread]