RFR: 8214533 IBM-29626C is required for AIX default charset

[Ichiroh Takiguchi]

Hello.

I'm very sorry. It's my fault.
EUC_JP class was moved to java.base module.
(sun.nio.cs.EUC_JP).

make/data/charsetmapping/stdcs-aix should have EUC_JP_LINUX and 
EUC_JP_Open.

Could you suggest me how I should provide new webrev files ?

Thanks,
Ichiroh Takiguchi


On 2019-01-28 17:03, Baesken, Matthias wrote:
> Hello,   seems  8214533   got pushed  recently  into  jdk/jdk.   Now
> we see build errors on AIX  ,  are they related to  this change ?
> 
> 
> /nb/rs6000_64/nightly/output-jdk-test/support/gensrc/jdk.charsets/sun/nio/cs/ext/EUC_JP_LINUX.java:63:
> error: Decoder is not public in EUC_JP; cannot be accessed from
> outside package
>     private static class Decoder extends EUC_JP.Decoder {
>                                                ^
> /nb/rs6000_64/nightly/output-jdk-test/support/gensrc/jdk.charsets/sun/nio/cs/ext/EUC_JP_LINUX.java:69:
> error: Encoder is not public in EUC_JP; cannot be accessed from
> outside package
>     private static class Encoder extends EUC_JP.Encoder {
>                                                ^
> /nb/rs6000_64/nightly/output-jdk-test/support/gensrc/jdk.charsets/sun/nio/cs/ext/EUC_JP_Open.java:65:
> error: Decoder is not public in EUC_JP; cannot be accessed from
> outside package
>     private static class Decoder extends EUC_JP.Decoder {
>                                                ^
> /nb/rs6000_64/nightly/output-jdk-test/support/gensrc/jdk.charsets/sun/nio/cs/ext/EUC_JP_Open.java:85:
> error: Encoder is not public in EUC_JP; cannot be accessed from
> outside package
>     private static class Encoder extends EUC_JP.Encoder {
> 
> Best regards, Matthias
> 
> 
> 
>> -----Original Message-----
>> From: ppc-aix-port-dev <ppc-aix-port-dev-bounces at openjdk.java.net> On
>> Behalf Of Ichiroh Takiguchi
>> Sent: Dienstag, 15. Januar 2019 01:51
>> To: Alan Bateman <Alan.Bateman at oracle.com>
>> Cc: build-dev <build-dev at openjdk.java.net>; ppc-aix-port-dev <ppc-aix-
>> port-dev at openjdk.java.net>; core-libs-dev at openjdk.java.net
>> Subject: Re: RFR: 8214533 IBM-29626C is required for AIX default 
>> charset
>> 
>> Hello Alan.
>> 
>> Could you review the fix again ?
>> 
>> Bug:    https://bugs.openjdk.java.net/browse/JDK-8214533
>> Change: https://cr.openjdk.java.net/~itakiguchi/8214533/webrev.01/
>> 
>> I added IBM29626C charset as standard way.
>> Please give any suggestion and question.
>> 
>> Thanks,
>> Ichiroh Takiguchi
>> IBM Japan, Ltd.
>> 
>> On 2018-12-14 18:58, Ichiroh Takiguchi wrote:
>> > Hello Alan.
>> >
>> > I opened JDK-8215333 for Charset filtering issue [1].
>> > I cannot wait until JDK-8215333 is closed.
>> > Is it possible to put IBM-29626C charset with standard way ?
>> >
>> > [1] https://bugs.openjdk.java.net/browse/JDK-8215333
>> >
>> > Thanks,
>> > Ichiroh Takiguchi
>> >
>> > On 2018-12-10 21:21, Ichiroh Takiguchi wrote:
>> >> Hello Roger, Magnus and Alan.
>> >> I may need to put alias information into charsets file.
>> >> stdcs-xxx cannot handle this information...
>> >>
>> >> Still AIX needs IBM-29626C charset for default encoding...
>> >>
>> >> I appreciate if you give me further suggestions.
>> >>
>> >> Thanks,
>> >> Ichiroh Takiguchi
>> >>
>> >> On 2018-12-10 20:50, Alan Bateman wrote:
>> >>> On 10/12/2018 11:01, Magnus Ihse Bursie wrote:
>> >>>> On 2018-12-07 21:20, Roger Riggs wrote:
>> >>>>> Hi,
>> >>>>>
>> >>>>> It is a nice feature that charsets are selected at build time using
>> >>>>> the stdcs-xxx files.
>> >>>>> This change breaks that pattern and embeds os specific information
>> >>>>> in more than one place.
>> >>>>> That does not seem like an improvement.  Is there any alternative?
>> >>>> I agree. Why is it not enough just to add it to stdcs-aix?
>> >>> My reading of the patch is that the "os" key is to avoid generating
>> >>> it
>> >>> on non-AIX platforms, it will otherwise end up in jdk.charsets on
>> >>> non-AIX platforms. The general direction is welcome but I think
>> >>> further work and discussion will be needed to get the right set of
>> >>> changes to support filtering in the build. It can probably be
>> >>> separated from the changes to add IBM-29626C to AIX's java.base.
>> >>>
>> >>> -Alan