Looks fine: approved. Brian
On Jul 25, 2016, at 10:22 AM, Phil Race <philip.r...@oracle.com> wrote: > Hi, > > Anyone able to approve this ? > I already pushed a subsequent fix that unfortunately *depends* on this fix. > So I need to get this one in ASAP > > -phil. > > On 07/21/2016 03:53 PM, Phil Race wrote: >> https://bugs.openjdk.java.net/browse/JDK-7175487 >> http://cr.openjdk.java.net/~prr/7175487/ >> >> As I note in the bug eval the probable issue here is that the file was found >> but rejected. >> I've made the code more forgiving by removing the test that the file have >> references to font files. >> The XLFD pattern for locating the files will work if they are missing (I did >> some manual testing). >> >> Also the reference to OpenSolaris here is obsolete. There's more clean up >> that could be done >> around that but for this fix I wanted to keep it about this Linux issue >> except for cleaning up >> the overall if (..) logic >> >> -phil. >