On 8/18/17 8:31 AM, Jan Lahoda wrote:
For the langtools change, I think we shouldn't remove the entries from
the include.list. This list defines which packages/classes should and
should not be available when compiling for previous versions of the
platform using --release. The removed entries are exclude entries, so if
removed, it would mean the classes should be available when compiling
for previous platform versions, which is not the intent, I think.
That makes sense.
Maybe adding a comment (or a JBS entry) that these should be cleaned up
when --release is no longer supporting 9?
Sure. I restored the entries and added the following comment:
#Some of the APIs below have been removed (see JDK-8159544). However, these
#need to be retained until --release no longer supports 9.
See also the updated webrev:
http://cr.openjdk.java.net/~mullan/webrevs/8159544/langtools.webrev.01/
--Sean
Thanks,
Jan
On 17.8.2017 21:08, Sean Mullan wrote:
Please review this JDK 10 change to remove the deprecated classes in
com.sun.security.auth.** that have been previously marked with
forRemoval=true in JDK 9.
webrev: http://cr.openjdk.java.net/~mullan/webrevs/8159544/
I have also copied Jan for reviewing a change in langtools, and also
build-dev for a change to one of the JDK Makefiles.
Thanks,
Sean
