Thanks for the review!
On 11/11/14 3:04 PM, serguei.spit...@oracle.com wrote:
Dan,
The fix looks good.
Thanks!
Nice cleanup from workarounds: Good Thing (TM)! :)
Yes, this has been in the queue for quite a while... :-)
Dan
Thanks,
Serguei
On 11/10/14 4:00 PM, Daniel D. Daugherty wrote:
Greetings,
I have a Solaris Full Debug Symbols (FDS) fix ready for review.
Yes, it is a small fix, but it is in Makefiles so feel free to
run screaming from the room... :-) On the plus side the fix does
delete two work around source files (Coleen would say that's a
Good Thing (TM)!)
The fix is to detect the version of GNU objcopy that is being
used on the machine and only enable Full Debug Symbols when that
version is 2.21.1 or newer. If you don't have the right version,
then the build drops back to pre-FDS build configs with a message
like this:
WARNING: /usr/sfw/bin/gobjcopy --version info:
WARNING: GNU objcopy 2.15
WARNING: an objcopy version of 2.21.1 or newer is needed to create
valid .debuginfo files.
WARNING: ignoring above objcopy command.
WARNING: patch 149063-01 or newer contains the correct Solaris 10
SPARC version.
WARNING: patch 149064-01 or newer contains the correct Solaris 10 X86
version.
WARNING: Solaris 11 Update 1 contains the correct version.
INFO: no objcopy cmd found so cannot create .debuginfo files.
INFO: ENABLE_FULL_DEBUG_SYMBOLS=0
This work is being tracked by the following bug IDs:
JDK-8033602 wrong stabs data in libjvm.debuginfo on JDK 8 - SPARC
https://bugs.openjdk.java.net/browse/JDK-8033602
JDK-8034005 cannot debug in synchronizer.o or objectMonitor.o on
Solaris X86
https://bugs.openjdk.java.net/browse/JDK-8034005
Here is the webrev URL:
http://cr.openjdk.java.net/~dcubed/8033602-webrev/0-jdk9-hs-rt/
Testing:
- JPRT test jobs to verify that the current JPRT Solaris hosts
are happy
- local builds on my Solaris 10 X86 machine to verify that the
wrong version of GNU objcopy is caught
Thanks, in advance, for any comments, questions or suggestions.
Dan
