The fix looks fine. I think it should go via the build forest, in the
expectation
that build is likely to integrate to master before AWT or 2D (which
would have been the second choice).
And its not a proprietary vs open thing, rather it seemed more efficient
to me to do both at once.
-phil.
Andrew John Hughes wrote:
2009/10/26 Phil Race <phil.r...@sun.com>:
Igor is also fixing the closed src ..
-phil.
Right, well obviously I can't fix the proprietary source code base by
definition. I don't think that's an acceptable reason for refusing
external contributions. If you want to continue to maintain a
proprietary version as well, then you need to be prepared to integrate
community contributions into it.
I've applied the following webrev to the AWT tree:
http://cr.openjdk.java.net/~andrew/6895287/webrev.01/
and it builds. It also works when applied to the IcedTea forest,
removing the original error I posted.
Just give the nod and I can push this.
Andrew John Hughes wrote:
2009/10/26 Igor Nekrestyanov <igor.nekrestya...@sun.com>:
Andrew,
i've filed 6895287 for this and plan to fix it soon.
(this will likely a bit faster in terms of "time to get fix into
master").
-igor
Thanks for filing a bug ID for this. I'm already working on a webrev
for this, so let's not duplicate the work. I'll post it as soon as it
builds.
On 10/26/09 11:19 AM, Andrew John Hughes wrote:
2009/10/26 Kelly O'Hair<kelly.oh...@sun.com>:
Is someone fixing this bootstrap build problem with
FontManager.getFontPath(true) and the fontchecker?
Not sure I care much how it gets fixed, I just need it fixed.
-kto
You and me both! I'll look at removing fontchecker.
