Looks good.<br><br>//Fredrik<br><br><div class="gmail_quote">2011/2/15 David Holmes <span dir="ltr"><<a href="mailto:David.Holmes@oracle.com">David.Holmes@oracle.com</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hi Kelly (and other build-dev folk)<br>
<br>
This CR has already been reviewed internally (Hi Paul!) but I updated to not "document" the new ALT_ variable as Paul suggested:<br>
<br>
<a href="http://cr.openjdk.java.net/%7Edholmes/7018429/webrev.02/" target="_blank">http://cr.openjdk.java.net/~dholmes/7018429/webrev.02/</a><br>
<br>
It's a trivial change to adjust the JDK_IMPORT_PATH if there is an ALT_JDK_TARGET_IMPORT_PATH<br>
<br>
Thanks,<br>
David<br>
<br>
-------- Original Message --------<br>
Subject: Request for review: 7018429 JPRT: Update Makefile to use ALT_JDK_TARGET_IMPORT_PATH for copying JDK<br>
Date: Thu, 10 Feb 2011 16:54:44 +1000<br>
From: David Holmes <<a href="mailto:David.Holmes@oracle.com" target="_blank">David.Holmes@oracle.com</a>><br>
Organization: Oracle Corporation<br>
<br>
Review mainly targeted at runtime and embedded folk but just wanted to<br>
give everyone a heads up.<br>
<br>
<a href="http://cr.openjdk.java.net/%7Edholmes/7018429/" target="_blank">http://cr.openjdk.java.net/~dholmes/7018429/</a><br>
<br>
Pretty straight-forward. If ALT_JDK_TARGET_IMPORT_PATH is set then we<br>
use that to set JDK_IMPORT_PATH.<br>
<br>
Tested with arm build and regular SE linux x86.<br>
<br>
Thanks,<br><font color="#888888">
David<br>
</font></blockquote></div><br>