RFR: 8214796: Create a jlink plugin for stripping debug info symbols from native libraries

[Magnus Ihse Bursie]

> 15 feb. 2019 kl. 18:01 skrev Severin Gehwolf <sgehwolf at redhat.com>:
> 
> Hi Alan, Mandy,
> 
> Here is the next round of this change:
> http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8214796/06/webrev/

Looks good from the build PoV, with one minor nit: In JtregNativeJdk.gmk, please use the new construct:

ifeq ($(call isTargetOs, linux), true)

/Magnus 

> 
> It now also has tests which don't require objcopy to be present on test
> machines as well as being integrated with --strip-debug (once JDK-
> 8218913 is pushed). --list-plugins help output for --strip-native-
> debug-symbols now reads:
> 
> ----->8-------------------------
> Plugin Name: strip-native-debug-symbols
> Option: --strip-native-debug-symbols=<omit-debuginfo|keep-debuginfo>
> Description: Strip debug symbols from native libraries (if any). 
> This plugin requires at least one option: 
>   objcopy: The path to the objcopy binary. Defaults to objcopy in PATH.
>   omit-debuginfo: Do not keep debug info files. Defaults to true.
>   keep-debuginfo[=<ext>]: Keep debug info files in <file>.<ext>.
>                           Defaults to <file>.debuginfo 
> Examples: --strip-native-debug-symbols objcopy=/usr/bin/objcopy
>           --strip-native-debug-symbols=omit-debuginfo
>           --strip-native-debug-symbols keep-debuginfo:objcopy=objcopy
> ----->8-------------------------
> 
> Testing: jdk.jlink tests jdk/submit (ongoing), added tests:
>         DefaultStripDebugPluginTest, StripNativeDebugSymbolsPluginTest
> 
> Note: IntegrationTest needed a heap size bump as this now runs
>      StripNativeDebugSymbolsPluginTest on some systems. Especially with
>      --with-native-debug-symbols=internal builds this can eat some
>      memory.
> 
> Please let me know what you think.
> 
> Thanks,
> Severin
> 
>> On Wed, 2019-02-13 at 15:36 -0800, Mandy Chung wrote:
>> On 2/13/19 1:04 AM, Severin Gehwolf wrote:
>>>>     --strip-native-debug-symbols no-keep-debuginfo
>>> --strip-native-debug-symbols strip-debuginfo
>>> --strip-native-debug-symbols remove-debuginfo
>>> 
>>> It would avoid using negation.
>> 
>> What about 'omit', i.e.
>> 
>> --strip-native-debug-symbols  omit-debuginfo
>> --strip-native-debug-symbols  keep-debuginfo=<ext>
>> 
>>> Sure. I've filed JDK-8218913 for doing this first. Would a name of --
>>> strip-java-debug-symbols be acceptable? My thinking is that it would
>>> have nice symmetry with --strip-native-debug-symbols. Thoughts?
>> 
>> --strip-native-debug-symbols is okay.
>> 
>> Having a second thought, javac -g and gcc -g both say "debugging
>> information".  I think we could go with:
>> 
>> --strip-java-debug-info
>> --strip-native-debug-info  omit-debuginfo
>> --strip-native-debug-info  keep-debuginfo=<ext>
>> 
>> unless anyone thinks `-debug-symbols` is better?
>> 
>> The options are getting pretty good now.
>> 
>> Mandy
>