Re: code review request for Full Debug Symbols Revamp (7102323, 7136506)

2012-03-17 Thread Daniel D. Daugherty
Thanks for the review. Replies embedded below... Dan, I've reviewed this: http://cr.openjdk.java.net/~dcubed/fds_revamp/7102323-webrev/1-hotspot-full/ Wow, you fixed two existing bugs in the make file: *make/solaris/makefiles/dtrace.make * -[ -f $(XLIBJVM_DB_G_DEBUGINFO) ] || { l

Re: code review request for Full Debug Symbols Revamp (7102323, 7136506)

2012-03-17 Thread serguei.spit...@oracle.com
Dan, I've reviewed this: http://cr.openjdk.java.net/~dcubed/fds_revamp/7102323-webrev/1-hotspot-full/ Wow, you fixed two existing bugs in the make file: *make/solaris/makefiles/dtrace.make * -[ -f $(XLIBJVM_DB_G_DEBUGINFO) ] || { ln -s $(LIBJVM_DB_DEBUGINFO) $(XLIBJVM_DB_G_DEB