On 8 Jun 2013, at 02:59, Joe Darcy <[email protected]> wrote:
> Approved.
+1
-Chris.
>
> -Joe
>
> On 6/7/2013 5:52 PM, Kumar Srinivasan wrote:
>> Hi,
>>
>> Can I get a quick review please, the fix excludes j2sdk-image/bin/jmc{.exe}
>> from the test, which invokes the tool and verifies that -version and
>> -Joptions work.
>> As the subject says invoking this tool with any option puts up a window,
>> hanging up
>> the test.
>>
>> Thanks
>> Kumar
>>
>>
>> diff --git a/test/tools/launcher/VersionCheck.java
>> b/test/tools/launcher/VersionCheck.java
>> --- a/test/tools/launcher/VersionCheck.java
>> +++ b/test/tools/launcher/VersionCheck.java
>> @@ -49,6 +49,7 @@
>> "javaw",
>> "javaws",
>> "jcontrol",
>> + "jmc",
>> "jvisualvm",
>> "packager",
>> "unpack200",
>> @@ -72,6 +73,7 @@
>> "jdeps",
>> "jinfo",
>> "jmap",
>> + "jmc",
>> "jps",
>> "jrunscript",
>> "jjs",
>
