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

[Jonathan Lu]

Hi Phil, Volker,

Thanks for your comments, but I noticed that JRE_LIB_FONTS has been
referred by the solaris fontconfig file,

jdk/make/data/fontconfig/solaris.fontconfig.properties

So I'm wondering if we can do this in the same way as Solaris platform.


Thanks
Jonathan



On Tue, May 13, 2014 at 12:47 AM, Phil Race <philip.race at oracle.com> wrote:

> 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
>>>>
>>>>
>>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/ppc-aix-port-dev/attachments/20140513/34e42243/attachment.html>