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 <[email protected]> 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 > >
