Looks good.
/Erik
On 2019-11-20 12:03, Mikael Vidstedt wrote:
I noticed that most of the configure options include the default value
in the help string (in brackets), so here’s an updated webrev which
does exactly that:
Webrev:
http://cr.openjdk.java.net/~mikael/webrevs/8234370/webrev.01/open/webrev
Webrev (incremental):
http://cr.openjdk.java.net/~mikael/webrevs/8234370/webrev.01.incr/open/webrev/
Cheers,
Mikael
On Nov 20, 2019, at 7:47 AM, Erik Joelsson <erik.joels...@oracle.com
<mailto:erik.joels...@oracle.com>> wrote:
Looks good.
/Erik
On 2019-11-18 15:28, Mikael Vidstedt wrote:
Please review this change which implements the changes for JEP 362:
Deprecate the Solaris and SPARC Ports.
JBS: https://bugs.openjdk.java.net/browse/JDK-8234370
Webrev:
http://cr.openjdk.java.net/~mikael/webrevs/8234370/webrev.00/open/webrev/
I’ve tested that the expected error message is produced by default,
and that running with --enable-deprecated-ports=yes produces a
warning instead.
Cheers,
Mikael