RFR 8190378: Java EE and CORBA modules removal
Lance Andersen
lance.andersen at oracle.com
Wed Feb 7 22:33:13 UTC 2018
Hi Mandy
> On Feb 7, 2018, at 5:00 PM, mandy chung <mandy.chung at oracle.com> wrote:
>
> Hi Lance,
>
> Great to see this JEP moving along. I reviewed all changes except
> test/langtools/tools/javac tests.
>
> Looks fine overall.
Thank you for the review.
> Minor comments:
>
> src/java.base/share/lib/security/default.policy
> - no change in this file.
Weird, not sure what happened, but it is now fixed.
>
> test/jdk/tools/jmod/hashes/HashesTest.java
> test/jdk/tools/launcher/modules/addexports/AddExportsTest.java
> - I think we should replace this test case with a different upgradeable module.
> It's okay to remove this case in this patch and follow up separately with
> a new JBS issue.
I can do that.
>
> test/langtools/tools/jdeps/modules/patches/java/sql/NonNull.java
> - copyright start year needs update.
Updated.
Best
Lance
>
> Mandy
>
> On 2/7/18 8:57 AM, Lance Andersen wrote:
>> Hi all,
>>
>> I think we are at a point where we are ready to start reviewing the changes to remove the Java EE and CORBA modules as JEP 320, JDK-8189188, has been targeted to JDK 11.
>> The CSR for removing the modules has been approved: https://bugs.openjdk.java.net/browse/JDK-8193757 <https://bugs.openjdk.java.net/browse/JDK-8193757> <https://bugs.openjdk.java.net/browse/JDK-8193757> <https://bugs.openjdk.java.net/browse/JDK-8193757>
>>
>> The open webrev can be found at: http://cr.openjdk.java.net/~lancea/8190378/open_changes/ <http://cr.openjdk.java.net/~lancea/8190378/open_changes/> <http://cr.openjdk.java.net/~lancea/8190378/open_changes/> <http://cr.openjdk.java.net/~lancea/8190378/open_changes/>
>>
>> To make the open review easier, I have broken the changes into 5 webrevs:
>> build changes are: http://cr.openjdk.java.net/~lancea/8190378/open_changes/build_webrev/ <http://cr.openjdk.java.net/~lancea/8190378/open_changes/build_webrev/> <http://cr.openjdk.java.net/~lancea/8190378/open_changes/build_webrev/> <http://cr.openjdk.java.net/~lancea/8190378/open_changes/build_webrev/>
>> miscellaneous changes are at: http://cr.openjdk.java.net/~lancea/8190378/open_changes/misc_webrev/ <http://cr.openjdk.java.net/~lancea/8190378/open_changes/misc_webrev/> <http://cr.openjdk.java.net/~lancea/8190378/open_changes/misc_webrev/> <http://cr.openjdk.java.net/~lancea/8190378/open_changes/misc_webrev/>
>> module changes are at: http://cr.openjdk.java.net/~lancea/8190378/open_changes/modules_webrev/ <http://cr.openjdk.java.net/~lancea/8190378/open_changes/modules_webrev/> <http://cr.openjdk.java.net/~lancea/8190378/open_changes/modules_webrev/> <http://cr.openjdk.java.net/~lancea/8190378/open_changes/modules_webrev/>
>> rmic changes are at: http://cr.openjdk.java.net/~lancea/8190378/open_changes/rmic_webrev/ <http://cr.openjdk.java.net/~lancea/8190378/open_changes/rmic_webrev/> <http://cr.openjdk.java.net/~lancea/8190378/open_changes/rmic_webrev/> <http://cr.openjdk.java.net/~lancea/8190378/open_changes/rmic_webrev/>
>> test changes are at: http://cr.openjdk.java.net/~lancea/8190378/open_changes/tests_webrev/ <http://cr.openjdk.java.net/~lancea/8190378/open_changes/tests_webrev/> <http://cr.openjdk.java.net/~lancea/8190378/open_changes/tests_webrev/> <http://cr.openjdk.java.net/~lancea/8190378/open_changes/tests_webrev/>
>>
>> As part of the removal, the following issues have also been logged:
>> Removal of the Java EE and CORBA tools from the documentation: https://bugs.openjdk.java.net/browse/JDK-8193906 <https://bugs.openjdk.java.net/browse/JDK-8193906> <https://bugs.openjdk.java.net/browse/JDK-8193906> <https://bugs.openjdk.java.net/browse/JDK-8193906>
>> Updating the RMIC man pages for the removal of the -iiop and -idl options: https://bugs.openjdk.java.net/browse/JDK-8196510 <https://bugs.openjdk.java.net/browse/JDK-8196510> <https://bugs.openjdk.java.net/browse/JDK-8196510> <https://bugs.openjdk.java.net/browse/JDK-8196510>
>> Hotspot tests may require further updating or just removed: https://bugs.openjdk.java.net/browse/JDK-8194310 <https://bugs.openjdk.java.net/browse/JDK-8194310> <https://bugs.openjdk.java.net/browse/JDK-8194310> <https://bugs.openjdk.java.net/browse/JDK-8194310>
>> jdeprescan will need updates due to the removal of the Java EE and CORBA modules: https://bugs.openjdk.java.net/browse/JDK-8194308 <https://bugs.openjdk.java.net/browse/JDK-8194308> <https://bugs.openjdk.java.net/browse/JDK-8194308> <https://bugs.openjdk.java.net/browse/JDK-8194308>
>>
>>
>>
>> Best,
>> Lance
>>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
>> Oracle Java Engineering
>> 1 Network Drive
>> Burlington, MA 01803
>> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com> <mailto:Lance.Andersen at oracle.com> <mailto:Lance.Andersen at oracle.com>
>>
>>
>>
>>
>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
More information about the core-libs-dev
mailing list