[Fwd: Re: Upcoming changes to JDK7 build process: JIBX]

[Andrew John Hughes]

2009/9/5 Joe Darcy <Joe.Darcy at sun.com>:
> Andrew John Hughes wrote:
>>
>> 2009/9/3 Andrew John Hughes <gnu_andrew at member.fsf.org>:
>>
>>>
>>> 2009/9/3 Joseph D. Darcy <Joe.Darcy at sun.com>:
>>>
>>>>
>>>> Andrew John Hughes wrote:
>>>>
>>>>>
>>>>> 2009/8/31 Joseph D. Darcy <Joe.Darcy at sun.com>:
>>>>>
>>>>>
>>>>>>
>>>>>> Without the JIBX build dependency, I would be happy for the Nimbus
>>>>>> work
>>>>>> to
>>>>>> be ported to OpenJDK 6.
>>>>>>
>>>>>> -Joe
>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>> The other immediate problem I see with this is that the OpenJDK
>>>>> version adds public API in the form of javax.swing.plaf.nimbus:
>>>>>
>>>>>
>>>>>
>>>>> http://builder.classpath.org/japi/openjdk7-openjdk6.html#pkg_javax_swing_plaf_nimbus
>>>>>
>>>>> This would need to be removed before Nimbus could be added to OpenJDK6.
>>>>>
>>>>>
>>>>
>>>> Yes, I'm not familiar with the details of how Nimbus was implemented in
>>>> the
>>>> 6 update release without java.* or javax.* API changes.
>>>>
>>>> -Joe
>>>>
>>>>
>>>
>>> I'll take a look either today or tomorrow and see if I can produce
>>> something from the changesets that would work for 6.  If the javax.*
>>> classes are just a layer on top of the Sun implementation, it should
>>> be easy enough to just remove them.  I'm guessing this means Nimbus is
>>> part of the spec. for JDK7.  If only such processes were more open...
>>> --
>>> Andrew :-)
>>>
>>> Free Java Software Engineer
>>> Red Hat, Inc. (http://www.redhat.com)
>>>
>>> Support Free Java!
>>> Contribute to GNU Classpath and the OpenJDK
>>> http://www.gnu.org/software/classpath
>>> http://openjdk.java.net
>>>
>>> PGP Key: 94EFD9D8 (http://subkeys.pgp.net)
>>> Fingerprint: F8EF F1EA 401E 2E60 15FA  7927 142C 2591 94EF D9D8
>>>
>>>
>>
>> Ok so it appears Nimbus was in com.sun.java.swing.plaf.nimbus for the
>> proprietary JDK6 release (there are backwards compatibility hooks
>> there instead in OpenJDK7).  So I just reversed that move for our
>> version.
>>
>> Here's a webrev for the first Nimbus patch:
>>
>> http://cr.openjdk.java.net/~andrew/6591875/webrev/jdk.patch
>>
>> This is just Nimbus as first imported with the JIBX requirement.  It's
>> already a big patch without bringing in all the Nimbus changes that
>> occurred afterwards (there are 10 further changesets)  My plan is to
>> backport each Nimbus-related changeset separately so that we
>> correspond as closely as possible with OpenJDK7 (and the same goes for
>> any pertinent future fixes).
>>
>> Does this look ok to push?  I've done a test build and was able to run
>> a simple Swing application using the Nimbus look and feel
>> (http://blog.fuseyism.com/index.php/2009/04/09/nimbus-arrives/)
>>
>> Cheers,
>>
>
> Hello.
>
> Monday is a holiday in the US so I won't be able to start looking at this
> until later next week.
>
> Thanks,
>
> -Joe
>

Ok, no problem.  Thanks for keeping me in the loop :)
-- 
Andrew :-)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Support Free Java!
Contribute to GNU Classpath and the OpenJDK
http://www.gnu.org/software/classpath
http://openjdk.java.net

PGP Key: 94EFD9D8 (http://subkeys.pgp.net)
Fingerprint: F8EF F1EA 401E 2E60 15FA  7927 142C 2591 94EF D9D8