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:35:17 UTC 2015


Oh - you should also remove the entries in ProblemList.txt.

> On 4 sep 2015, at 12:34, Staffan Larsen <staffan.larsen at oracle.com> wrote:
> 
> 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