Fwd: code review request for initial JDK FDS support (7071907)
keith.mcguigan at oracle.com
Tue Apr 10 13:54:40 UTC 2012
I think it looks good. The new form of the 'find' commands in Release.gmk could
use a comment, though. It took me a few minutes to realize that EXE_SUFFIX was
empty for linux/solaris -- it looks like there's a redundancy in the logic (both
NOT name *.debuginfo AND name *.exe). I suppose there is a redundancy, but
only for windows; that new clause is needed only for linux/solaris.
Daniel D. Daugherty wrote:
> Coming soon to a JDK repo near you! Full Debug Symbols!
> OK, to just a subset of libraries and programs... on Linux and Solaris...
> If you're a Windows fan, the JDK repo has had Full Debug Symbols support
> since way back in JDK1.4.1... Now we're trying get Linux and Solaris
> caught up...
> Runtime Team, we don't have much in the JDK repo, but I tried to cover
> our few libraries and programs. Let me know if I missed anything...
> Serviceability Team, all of your demos, libraries and programs are
> covered... for some reason, updating those seemed like reliving old
> times and I didn't think you'd mind... :-)
> Here is the webrev URL:
> Thanks, in advance, for any review comments.
> For those of you that are keeping track of all the FDS
> changesets, not everything has hit the various master
> repos yet. As a reminder, FDS has to hit the closed
> install repo first. The open root and jdk repos along
> with the closed deploy repo are in the second wave. And
> the hotspot repo, being more Mercurial than his fellow
> ghosts, will make his appearance in his own good time
> (and via a different set of repos)...
> Apologies to Dickens, of course... :-)
More information about the build-dev