Request for review 6237353: Remove sun.io package from j2se binary
Alexandre Boulgakov
alexandre.boulgakov at oracle.com
Wed Aug 17 18:34:03 UTC 2011
Hello Sherman,
You would also need to remove "JAVAC_LINT_OPTIONS =
-Xlint:all,-deprecation" on line 35. When JAVAC_MAX_WARNINGS=true,
JAVAC_LINT_OPTIONS automatically gets set to -Xlint:all [1], so that
line will not do anything.
Thanks,
Sasha
[1] Actually, it gets set to "-Xlint:all -Xlint:-path", but that's
temporary. This happens in jdk/make/common/shared/Defs-java.gmk.
On 8/17/2011 11:41 AM, Xueming Shen wrote:
> Thanks Sasha,
>
> The flag has been set to true, as suggested.
>
> The webrev has been updated according.
>
> http://cr.openjdk.java.net/~sherman/6237353/webrev/
> <http://cr.openjdk.java.net/%7Esherman/6237353/webrev/>
>
> -Sherman
>
> On 08/17/2011 05:54 AM, Alexandre Boulgakov wrote:
>> Hello Sherman,
>>
>> Can you check if any deprecation warnings are generated from the
>> other classes built by java/sun_nio/Makefile? Suppressing deprecation
>> warnings might no longer be necessary now that sun.io is gone, and
>> then you can change lines 34-35 in the makefile to
>> "JAVAC_MAX_WARNINGS = true".
>>
>> Cheers,
>> Sasha
>>
>> On 8/17/2011 2:07 AM, Alan Bateman wrote:
>>> Xueming Shen wrote:
>>>> Hi,
>>>>
>>>> This is something long over due. Some background info.
>>>>
>>>> (1)The java.nio.charset package (to replace the private sun.io
>>>> package) is added into
>>>> JDK 1.4 as the result of the nio JSR. Part of the sun.io
>>>> converters were migrated to
>>>> the java.nio.charset implementation in JDK8.
>>>> (2) All supported charsets/converters were migrated to
>>>> java.nio.charset in JDK5. As the
>>>> result no more bug fix/rfe maintenance work for sun.io since
>>>> then.
>>>> (3) Removed all sun.io dependencies (internal sun.io usage inside
>>>> JDK) from the JDK
>>>> repository at the beginning of JDK6.
>>>> (4) The sun.io package was removed from the JDK runtime at JDK6 b29.
>>>>
>>>> But we then had one important licensee strongly asked to undo the
>>>> removal. The sun.io
>>>> was putback in JDK6 6u59/beta.
>>>>
>>>> Now let's try this again in JDK8, the licensee that was against it
>>>> in JDK6 last time has been
>>>> communicated and is OK with the plan this time. And the JDBC EG
>>>> group (which heavily
>>>> used sun.io package in the past) is/will be noticed as well. We
>>>> assume after 3 major releases
>>>> the risk of breaking someone's application (as the result of this
>>>> change) is limited.
>>>>
>>>> Here is the webrev. Since we have tried it already (the workspace
>>>> had been cleaned up
>>>> already) in jdk6 again already, the change this time is relatively
>>>> simple, mainly only 2 makefile
>>>> changes. The rest is the "deleting" and several minor changes in
>>>> testing.
>>>>
>>>> http://cr.openjdk.java.net/~sherman/6237353/webrev/
>>> Thanks for doing this, and thanks Lance for checking with all the
>>> JDBC folks in your phone book.
>>>
>>> The changes looks good to me. Push it quick and run :-)
>>>
>>> -Alan
>>>
>
More information about the core-libs-dev
mailing list