Hi!

Please review this tiny patch which removes the special casing of 
`--with-native-debug-symbols=external` for the static libs build. I don't see 
why this is needed. If no debug symbols are wanted 
`--with-native-debug-symbols=none` can be used to achieve the same effect. 
Therefore, I propose to remove this hunk.

Testing: Inspecting of the log files and seeing that `-g` switch is there. Run 
the reproducer test on the resulting static libraries.

Thoughts?

-------------

Commit messages:
 - 8271148: static-libs-image target --with-native-debug-symbols=external 
doesn't produce debug info

Changes: https://git.openjdk.java.net/jdk/pull/4876/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=4876&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8271148
  Stats: 9 lines in 1 file changed: 0 ins; 9 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4876.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4876/head:pull/4876

PR: https://git.openjdk.java.net/jdk/pull/4876

Reply via email to