Request for code review

Kelly O'Hair kelly.ohair at
Thu Sep 9 15:49:52 UTC 2010

The change looks fine, but I suspect that between the time you push  
this change, and a promoted jdk7 build is created from it,
anyone building the jdk repository by itself will fail because the  
imported jdk7 image won't have these files.
A bit of a catch22.

Either the change needs to include some kind of check for existence:

ifneq ($(wildcard $(LIB_LOCATION)/$(SERVER_LOCATION)/$ 

which could be removed after the promotion.

Or you need to time this change so it happens very shortly before the  


On Sep 9, 2010, at 6:24 AM, Keith McGuigan wrote:

> Hello,
> I'd like a code review for this change which imports the  
> and files from Hotspot into the  
> solaris JDK images (libjvm_dtrace was missing, and libjvm_db was  
> missing for x86).
> Thanks!
> --
> - Keith

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the build-dev mailing list