Request for code review
kelly.ohair at oracle.com
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:
> I'd like a code review for this change which imports the
> libjvm_db.so and libjvm_dtrace.so files from Hotspot into the
> solaris JDK images (libjvm_dtrace was missing, and libjvm_db was
> missing for x86).
> - Keith
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the build-dev