RFR 8134420: sun/tools/jps/TestJpsClass fails with java.lang.RuntimeException: The line 'line 2' does not match pattern '^\\d+\\s+.*': expected true, was false

Staffan Larsen staffan.larsen at oracle.com
Fri Sep 4 10:34:42 UTC 2015


Good!

> On 4 sep 2015, at 12:21, Jaroslav Bachorik <jaroslav.bachorik at oracle.com> wrote:
> 
> On 2.9.2015 20:44, Staffan Larsen wrote:
>> Looks good, but needs a test.
> 
> I've modified the 'jps' generic test to exercise multi-lined properties.
> 
> http://cr.openjdk.java.net/~jbachorik/8134420/webrev.01
> 
> -JB-
> 
>> 
>>> On 2 sep 2015, at 20:21, Jaroslav Bachorik <jaroslav.bachorik at oracle.com> wrote:
>>> 
>>> Please, review the following change
>>> 
>>> Issue : https://bugs.openjdk.java.net/browse/JDK-8134420
>>> Webrev: http://cr.openjdk.java.net/~jbachorik/8134420/webrev.00
>>> 
>>> Currently, 'jps' will print multi-line vm args as they are - eg. splitting the output into multiple lines. This kind of output is not very easy to read and, incidentally, it can break this test.
>>> 
>>> My proposal is to replace all '\n' and '\r' characters with "\n" and "\r" strings respectively to keep the output format being 'one line per one JVM'.
>>> 
>>> Thanks,
>>> 
>>> -JB-
>> 
> 



More information about the serviceability-dev mailing list