7121110 : JAXP 1.4.5 update 1 for 7u4 & jaxp source in jdk7 repo

[Kelly O'Hair]

In terms of monitoring the changes made to the jaxp sources, being in it's own repository makes that easy.
You can easily scan the changesets made to the repository and see when strange changesets appear at
  http://hg.openjdk.java.net/jdk8/jdk8/jaxp

If we folded the jaxp sources into the jdk repository, that becomes harder, but has other benefits which will
probably become more apparent as the jigsaw project gets integrated.

The other issue is how dependent the jaxp sources become on newer features in the jdk8 javac or apis.
Historically, your jaxp sources had to stay back on the older Java api and language, not sure how that
will change going forward, or if it will. Again, that's a little easier when it's part of the jdk repository where
the new apis are in the same repository. But we can deal with that when the time comes.

-kto

On Dec 27, 2011, at 10:21 AM, Joe Wang wrote:

> Thanks Kelly!
> 
> Indeed, that was the original intention when you designed the bundle process. Besides the unfortunate download problem, it actually works great in terms of guaranteeing that we have one single editable source.
> 
> Next, maybe we could start to work on migrating the jaxp project source into the jdk. But it's the holiday season, we don't have to think too hard :). In the meantime, we just have to make sure patches are applied to the jaxp project first (through JIRA report for example) since that's how our dev/test process is set up.
> 
> --Joe
> 
> On 12/24/2011 8:56 AM, Kelly O'Hair wrote:
>> Looks great to me.
>> 
>> There is still the issue of making sure the right people (like you :^) are the ones patching this code, so we know
>> what we have matches the public JAXP project.
>> 
>> -kto
>> 
>> On Dec 23, 2011, at 1:27 PM, Joe Wang wrote:
>> 
>>> Hi All,
>>> 
>>> We have prepared a jaxp update for 7u4. The listed changes have been posted to the jdk7u-dev alias with "7u4 Request for approval for CR 7121110 - JAXP 1.4.5 update 1 for 7u4".
>>> 
>>> With this update, we are adding jaxp sources back into the jdk7 repository, which means we are abandoning the jaxp bundle download process. This is a significant change. We'd like to hear if you have any concerns.  If no objection, we'd like to push the change within the next week.
>>> 
>>> The webrev is here:  http://cr.openjdk.java.net/~joehw/jaxp145u1-changeset/webrev/
>>> 
>>> Thanks,
>>> Joe