Review request for 8034218: AIX: Provide a better fontconfig.properties file

[Volker Simonis]

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
>
>