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


Reply via email to