RFR(S): JDK-8032662 : test/compiler/ciReplay/TestSA.sh should report ulimit issues

[Staffan Larsen]

That sounds like a good idea.

new webrev: http://cr.openjdk.java.net/~sla/8032662/webrev.01/

/Staffan

On 24 jan 2014, at 16:25, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:

> I'm not sure that all system returns 'unlimited', some of them may return '-1', so I'd prefer '$new_ulimit != "unlimited" -a $new_ulimit !=  "-1"' as the condition.
> 
> In the rest, looks good.
> 
> Igor
> 
> On 01/24/2014 07:22 PM, Vladimir Kozlov wrote:
>> Looks fine to me but I want Igor's opinion as well since he is the author.
>> 
>> Thanks,
>> Vladimir
>> 
>> On 1/24/14 7:09 AM, Staffan Larsen wrote:
>>> Please review this small fix to a test to provide additional
>>> diagnostics on failures.
>>> 
>>> webrev: http://cr.openjdk.java.net/~sla/8032662/webrev.00/
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8032662
>>> 
>>> Thanks,
>>> /Staffan
>>>