[8u] RFA: 8036003: Add --with-native-debug-symbols=[none|internal|external|zipped]

[Rob McKenna]

Thanks, approved. Please add the noreg-build label to the bug.

    -Rob

On 19/07/18 10:11, Severin Gehwolf wrote:
> Hi Rob,
> 
> On Thu, 2018-07-19 at 00:19 +0100, Rob McKenna wrote:
> > Can you please send a separate enchancement backport approval
> > request?
> 
> Done:
> http://mail.openjdk.java.net/pipermail/jdk8u-dev/2018-July/007695.html
> 
> Thanks,
> Severin
> 
> > http://openjdk.java.net/projects/jdk8u/enhancement-template.html
> > 
> > On 18/07/18 15:58, Severin Gehwolf wrote:
> > > Hi,
> > > 
> > > Please approve this enhancement backport for JDK 8u. It adds a new
> > > configure option:
> > > --with-native-debug-symbols=[none|internal|external|zipped]
> > > 
> > > Such an option exists in JDK 9+ already and we've been using the --
> > > with-native-debug-symbols=internal switch for our distribution OpenJDK
> > > builds ever since. The backport was done in a way so that old behaviour
> > > keeps working. For example, if the new configure flag is missing this
> > > should be a no-op for existing build configs. However, if the new
> > > option is present, it'll use --with-native-debug-symbols, overriding
> > > old configure flags --disable-debug-symbols and --disable-zip-debug-
> > > info. This enhancement also fixes bug JDK-8207234[1].
> > > 
> > > bug: https://bugs.openjdk.java.net/browse/JDK-8036003
> > > webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8036003/webrev.02/
> > > review-thread: http://mail.openjdk.java.net/pipermail/build-dev/2018-July/022686.html
> > > 
> > > Thanks,
> > > Severin
> > > 
> > > [1] https://bugs.openjdk.java.net/browse/JDK-8207234