RFR: 8234370: Implementation of JEP 362: Deprecate the Solaris and SPARC Ports

Mikael Vidstedt mikael.vidstedt at oracle.com
Fri Nov 22 19:01:02 UTC 2019


Erik/Tim, thanks for the reviews!

On more small change: Update building.md (and .html) to reflect the deprecation of the ports:

http://cr.openjdk.java.net/~mikael/webrevs/8234370/webrev.02.incr/open/webrev/ <http://cr.openjdk.java.net/~mikael/webrevs/8234370/webrev.02.incr/open/webrev/>

Let me know if you think of other places where this should be reflected.

Cheers,
Mikael

> On Nov 22, 2019, at 6:17 AM, Tim Bell <tim.bell at oracle.com> wrote:
> 
> Mikael:
> 
> Looks good to me as well.
> 
> Tim
> 
> On 2019-11-22 06:10, Erik Joelsson wrote:
>> 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.joelsson at oracle.com <mailto:erik.joelsson at 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
>>>>> 
>>> 
> 




More information about the build-dev mailing list