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 <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/ <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> 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 >>