<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
  <title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
On 05/24/10 03:58 PM, Weijun Wang wrote:
<blockquote cite="mid:2D9B2FA5-80C4-403D-B5CF-146D007C5F0B@Sun.COM"
 type="cite">
  <blockquote type="cite">
    <blockquote type="cite">
      <pre wrap=""> 
      </pre>
      <blockquote type="cite">
        <pre wrap="">For test/sun/security/tools/jarsigner/diffend.sh, I strongly encourage this to be rewritten as a Java program.
   
        </pre>
      </blockquote>
      <pre wrap="">The test calls zip, unzip and jarsigner commands. It might be possible to convert all calls into Java methods but the result will be quite complicated and not easy to read. Almost all security tools tests are written as shell scripts.
 
      </pre>
    </blockquote>
    <pre wrap="">In langtools, we've taken to writing such code in Java using Runtime.exec; I recommend considering this approach for other shell tests too.
    </pre>
  </blockquote>
  <pre wrap=""><!---->
I'll consider it later.

  </pre>
  <blockquote type="cite">
    <pre wrap="">Is

6954621: small error in 6948909

also needed in OpenJDK 6?  If so, I approve all three bugs going back.
    </pre>
  </blockquote>
  <pre wrap=""><!---->
I've merged the code change in 6954621 into 6948909 for both 6u21 and 6-open. 6948909 is the main bug id anyway.
  </pre>
</blockquote>
<br>
Okay; but for bug tracking it is often easier if the bugs in question
have a sub CR (release specific entry) for each release it is going
into.&nbsp; In particular, it would ease future bug archeology efforts if
6954621 had a sub CR for OpenJDK 6.<br>
<br>
I approve the current versions of your changes going back OpenJDK 6 has
sub CRs for all three bugs.<br>
<br>
Thanks,<br>
<br>
-Joe<br>
</body>
</html>