URGENT: Request for approval for CR #7049963 -, DISTINGUISHED NAMES FOR CERT ARE ESCAPED IN JROCKIT 1.6(NOT COMPATIBLE, WITH JROC

[Mala Bankal]

HI Dalibor,

For jdk8 i have not added the Copyright , but for jdk7u2 i did. This happened during final push.
This is the reason why :

  
Initially  i had updated the copyright (to1996, 2011) for both jdk 8 and 7u2.
But then during merge found that there was some other fix which had updated the jdk8 file which holds my changes.
Obviously he/she updated the Copyright to the latest (to 1996, 2011).
I had to do a fresh clone since the hg merge did not work, it got complicated due to changes to the same file.
  
So i did a clone of jdk8 and re-applied the fix excluding the copyright.
If you compare jdk8 copyright changes (which have been putback) and 7u2 (which i have made) they are identical.
So i need not change the copyright for 7u2.
  
Not sure if Sean can review them before the deadline.

rgds
mala



On 9/15/2011 2:30 PM, Dalibor Topic wrote:
> On 9/15/11 10:49 AM, Mala Bankal wrote:
>> HI Dalibor,
>>
>> a) I can't see the bug on bugs.sun.com - please describe what this
>> change is supposed to fix.
>>
>> 6.0 and 5.0 introduce extract escape character.
>> for e.g if CN = "Qpass Inc"
>> older jdk 5.0 and 6.0 keytool prints CN = "Qpass Inc"
>> later ones print CN =  "\"Qpass Inc\""
>> Behaviour introduced due to Bug # 6927152.
>> Fix is to check if string is already quoted, don't re-quote
> Great, thanks.
>
>> b) Is the changeset athttp://hg.openjdk.java.net/jdk8/tl/jdk/rev/52bc200b14e5  the same patch as
>> inhttp://cr.openjdk.java.net/~mbankal/7049963/webrev.00/?
>>
>> Yes
> Unfortunately, not quite - there is a change in the copyright block in your webrev,
> that's not present in the changeset that was pushed into jdk8. So I have to ask Sean
> to take another look at it per our code review process. See the Project home page for
> details.
>
> cheers,
> dalibor topic