RFR: JDK-8036003: Add --with-debug-symbols=[none|internal|external|zipped]

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Fri Dec 11 11:52:04 UTC 2015


On 2015-12-11 00:47, Yasumasa Suenaga wrote:
> Hi build folks,
>
> Could you review it?
>
>>> http://cr.openjdk.java.net/~ysuenaga/JDK-8036003/webrev.06/

I like the new way of passing a help text for deprecation. Just a note, 
in the help text you called the new option "--with-debug-symbols" 
instead of "--with-native-debug-symbols", which is kind of unhelpful. 
;-) You can just fix this, no need to upload a new webrev.

So, if you just fix this, I'm okay with the patch.

Thank you for fighting this all the way through. I do apologize for the 
slow process.

/Magnus



More information about the build-dev mailing list