RFR: 8049229 sun/tools/jps/jps-l_1.sh fails with Execution failed: exit code 1

2014-07-03 Thread Staffan Larsen 
bug: https://bugs.openjdk.java.net/browse/JDK-8049229
webrev: http://cr.openjdk.java.net/~sla/8049229/webrev.00/

The problem here is that the awk scripts in the test match the lines in the ups 
output that end in “.jar twice and end up failing. The solution is to stop 
matching after we have found one match.

Thanks,
/Staffan

Re: RFR: 8049229 sun/tools/jps/jps-l_1.sh fails with Execution failed: exit code 1

2014-07-03 Thread Dmitry Samersoff 
Looks good for me.

On 2014-07-03 13:22, Staffan Larsen wrote:
 bug: https://bugs.openjdk.java.net/browse/JDK-8049229
 webrev: http://cr.openjdk.java.net/~sla/8049229/webrev.00/
 
 The problem here is that the awk scripts in the test match the lines in the 
 ups output that end in “.jar twice and end up failing. The solution is to 
 stop matching after we have found one match.
 
 Thanks,
 /Staffan
 


-- 
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.

Re: RFR: 8049229 sun/tools/jps/jps-l_1.sh fails with Execution failed: exit code 1

2014-07-03 Thread Jaroslav Bachorik 

Looks fine.

Those simple file names really look like FQNs ...

-JB-

On 07/03/2014 11:22 AM, Staffan Larsen wrote:

bug: https://bugs.openjdk.java.net/browse/JDK-8049229
webrev: http://cr.openjdk.java.net/~sla/8049229/webrev.00/

The problem here is that the awk scripts in the test match the lines in the ups 
output that end in “.jar twice and end up failing. The solution is to stop 
matching after we have found one match.

Thanks,
/Staffan