URGENT: Request for approval for CR #7049963 -, DISTINGUISHED NAMES FOR CERT ARE ESCAPED IN JROCKIT 1.6(NOT COMPATIBLE, WITH JROC (original) (raw)
Dalibor Topic dalibor.topic at oracle.com
Thu Sep 15 04:36:23 PDT 2011
- Previous message: URGENT: Request for approval for CR #7049963 -, DISTINGUISHED NAMES FOR CERT ARE ESCAPED IN JROCKIT 1.6(NOT COMPATIBLE, WITH JROC
- Next message: hg: jdk7u/jdk7u-dev/jdk: 7049963: DISTINGUISHED NAMES FOR CERT ARE ESCAPED IN JROCKIT 1.6(NOT COMPATIBLE WITH JROC
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 9/15/11 1:29 PM, Mala Bankal wrote:
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.
No need to, your explanation is perfectly fine. You can use me (robilad) as the reviewer - approved!
cheers, dalibor topic
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
-- Oracle <http://www.oracle.com> Dalibor Topic | Java F/OSS Ambassador Phone: +494023646738 tel:+494023646738 | Mobile: +491772664192 tel:+491772664192 Oracle Java Platform Group
ORACLE Deutschland B.V. & Co. KG | Nagelsweg 55 | 20097 Hamburg
ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 München Registergericht: Amtsgericht München, HRA 95603
Komplementärin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Geschäftsführer: Jürgen Kunz, Marcel van de Molen, Alexander van der Ven
Green Oracle <http://www.oracle.com/commitment> Oracle is committed to developing practices and products that help protect the environment
- Previous message: URGENT: Request for approval for CR #7049963 -, DISTINGUISHED NAMES FOR CERT ARE ESCAPED IN JROCKIT 1.6(NOT COMPATIBLE, WITH JROC
- Next message: hg: jdk7u/jdk7u-dev/jdk: 7049963: DISTINGUISHED NAMES FOR CERT ARE ESCAPED IN JROCKIT 1.6(NOT COMPATIBLE WITH JROC
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]