RFR(S) Solaris Full Debug Symbols (FDS) fix for 8033602 and 8034005

Daniel D. Daugherty daniel.daugherty at oracle.com
Thu Nov 13 18:18:40 UTC 2014


Magnus,

Thanks for the review!

Replies embedded below...

On 11/13/14 7:44 AM, Magnus Ihse Bursie wrote:
> On 2014-11-11 01:00, 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)!)
>
> ... but you're only deleting the make files?

Good catch! Looks like when I resurrected this fix from my JDK8
queue I missed a couple of deletes.


> src/os/solaris/add_gnu_debuglink/add_gnu_debuglink.c and 
> src/os/solaris/fix_empty_sec_hdr_flags/fix_empty_sec_hdr_flags.c could 
> be deleted as well, right?

Yes, these should be deleted and I'll do that in this fix.
Since these are two deletes of files that can no longer be
built anyway, I presume I don't need to sent out another
webrev...


>
> Good idea for the fix, anyway. I opened 
> https://bugs.openjdk.java.net/browse/JDK-8064808 to implement a 
> similar solution in configure.

Sounds good to me.

Dan


>
> /Magnus




More information about the build-dev mailing list