RFR: 8233291: [TESTBUG] tools/jlink/plugins/VendorInfoPluginsTest.java fails with debug VMs

[Mandy Chung]

On 10/31/19 9:08 PM, Jie Fu wrote:
> Hi Mandy,
>
> Thanks for your review and nice suggestion.
> Updated: http://cr.openjdk.java.net/~jiefu/8233291/webrev.01/
>

Why do you need to check if it contains "debug"?  I would expect it's 
simply empty string or the "jdk.debug" system property value appended 
with a space.

In any case, I'm okay with the generic pattern ".*build.*$" (webrev.00) 
if preferred.

> I had added '@requires vm.flavor == "server"' since the test is only 
> suitable for server VMs.
>

I think the test is not restricted for server VM.   Is it because of the 
"Server VM" check?  Should that check be fixed instead?

[1] 
http://hg.openjdk.java.net/jdk/jdk/file/tip/src/java.base/share/classes/java/lang/VersionProps.java.template#l217

Mandy