[OpenJDK 2D-Dev] <AWT Dev> Review request for 8034218: AIX: Provide a better fontconfig.properties file

[Phil Race]

removed awt-dev ...

 > I don't think we should reference the fonts
 > under $JRE_LIB_FONTS/ in the OpenJDK version of fontconfig.properties

Volker is correct on this point.

Since OpenJDK can't ship those fonts we made sure they were never used
in any openjdk fontconfig file.

-phil.


On 05/12/2014 09:42 AM, Phil Race wrote:
> All,
>
> If you want to continue this discussion i should be restarted on 2d-dev
> and awt-dev removed. AWT has nothing to do with the fonts ..
> I've fixed the bug sub-cat.
>
> -phil.
>
> On 05/12/2014 09:06 AM, Volker Simonis wrote:
>> Hi Jonathan,
>>
>> thanks for submitting this patch.
>>
>> Overall it looks good, but I don't think we should reference the fonts
>> under $JRE_LIB_FONTS/ in the OpenJDK version of fontconfig.properties
>> (at least not until IBM will not also contribute the corresponding
>> fonts :)
>>
>> I would also find it helpful to keep a reworked version of the comment:
>>
>>    27 # Minimal version for AIX using the standard Latin Type1 Fonts 
>> from the
>>    28 # package X11.fnt.iso_T1. These fonts are installed by default 
>> into
>>    29 # "/usr/lpp/X11/lib/X11/fonts/Type1" and sym-linked to
>> "/usr/lib/X11/fonts/Type1"
>>
>> which mentions which AIX packages provide the other referenced font 
>> files.
>>
>> Thank you and best regards,
>> Volker
>>
>>
>>
>> On Mon, May 12, 2014 at 12:37 PM, Jonathan Lu 
>> <luchsh at linux.vnet.ibm.com> wrote:
>>> Hi awt-dev,
>>>
>>> Please review the fix for issue:
>>> https://bugs.openjdk.java.net/browse/JDK-8034218
>>>
>>> The changes are placed at
>>> http://cr.openjdk.java.net/~luchsh/JDK-8034218/
>>>
>>> This patch is trying to provide a better fonctconfig.properties file 
>>> for AIX
>>> platform,
>>> which contains support for more languages, like CJK.
>>>
>>>
>>> Thanks
>>> Jonathan
>>>
>>>
>