DBus font installs [incomplete]

Deepak Bhole dbhole at redhat.com
Fri Apr 15 07:31:08 PDT 2011


* Dr Andrew John Hughes <ahughes at redhat.com> [2011-04-14 18:48]:
> On 15:32 Thu 14 Apr     , Deepak Bhole wrote:
> > > +         } else {
> > > +diff -r 9d90acd9f99f src/solaris/native/sun/awt/fontpath.c
> > > +--- openjdk/jdk/src/solaris/native/sun/awt/fontpath.c	Wed Feb 16 09:39:35 2011 -0500
> > > ++++ openjdk/jdk/src/solaris/native/sun/awt/fontpath.c	Wed Mar 02 09:24:59 2011 -0500
> > > +@@ -517,11 +517,11 @@
> > > + }
> > > + 
> > > + JNIEXPORT jstring JNICALL Java_sun_font_FontManager_getFontPath
> > > +-(JNIEnv *env, jclass obj, jboolean noType1) {
> > > ++(JNIEnv *env, jclass obj, jboolean noType1, jboolean recheck) {
> > 
> > We should find some other way to recheck (whatever it is doing).
> > Changing a public api, even for a sun.* class is a no-no.
> > 
> 
> For sun.*, it's possible.  I'd prefer it to some convoluted workaround.
> 

Wouldn't that make OpenJDK less compatible with (admittedly bad) code
that relies on sun.* API being a certain way?

Deepak



More information about the distro-pkg-dev mailing list