Hi, Mandy. Thanks for looking at this.
> We will need to find a home for this jdk.* API since java.base only > exports Java SE API per the design principles in JEP 200 The Modular > JDK. It’s fine to file an issue to follow up this. I've filed this bug and assigned it to myself: 8144062: Determine appropriate module for jdk.Version and jdk.OracleVersion https://bugs.openjdk.java.net/browse/JDK-8144062 > You will also need to find a docs page to publish this new API as well > (the current way is a page linked from the layer cake and this may be > changed). Yes, I'm aware of these discussions. Since I suspect that they won't be resolved within the next week or so, I've filed this bug: 8144069: Determine correct publication for jdk.Version and jdk.OracleVersion APIs https://bugs.openjdk.java.net/browse/JDK-8144069 Regards, iris -----Original Message----- From: Mandy Chung Sent: Wednesday, November 25, 2015 8:48 AM To: Iris Clark Cc: core-libs-dev@openjdk.java.net; verona-...@openjdk.java.net Subject: Re: RFR: 8072379: Implement jdk.Version and jdk.OracleVersion > On Nov 24, 2015, at 5:54 PM, Iris Clark <iris.cl...@oracle.com> wrote: > > Hi. > > Please review the new classes jdk.Version and jdk.OracleVersion. > These are simple Java APIs to parse, validate, and compare version numbers. > > Webrev > > http://cr.openjdk.java.net/~iris/verona/8072379/webrev.1/ This looks good to me. Alan raises a good question whether jdk.OracleVersion is intended to be pushed to OpenJDK. We will need to find a home for this jdk.* API since java.base only exports Java SE API per the design principles in JEP 200 The Modular JDK. It’s fine to file an issue to follow up this. W.r.t. the docs build, jdk is listed in top/make/common/NON_CORE_PKGS.gmk but there is no target in make/Javadoc.gmk building it. You will also need to find a docs page to publish this new API as well (the current way is a page linked from the layer cake and this may be changed). Mandy
