TEST_BUG: com/sun/jdi/BreakpointWithFullGC.sh fails to cleanup in Cygwin

[Daniel D. Daugherty]

Yes, "/dev/null" works with the "1.5.X" version of Cygwin.
1.5.X is considered the "legacy" version of Cygwin and I
believe the VM/SQE is still using it in the nightly testing
infrastructure.

Dan

547 2011.08.09 10:22:32 $ uname -a
CYGWIN_NT-5.1 oliver-lt 1.5.25(0.156/4/2) 2008-06-12 19:34 i686 Cygwin

548 2011.08.09 10:22:41 $ ls -l /dev/null
crw-rw-rw- 1 dcubed None 1, 3 Nov 30  2006 /dev/null

549 2011.08.09 10:22:45 $ echo "hello" > /dev/null

550 2011.08.09 10:22:52 $



On 8/9/11 10:18 AM, Jim Holmlund wrote:
> Will this work on older versions of cygwin?  Or do we have a spec 
> someplace that says cygwin versions older than xxxx aren't supported?
> - jjh
>
>
> On 8/9/2011 8:56 AM, Chris Hegarty wrote:
>> Sorry Dan, I'll put a more complete description in the bug report.
>>
>> This issue, from what I can tell only happens with very recent Cygwin 
>> versions. I'm running 1.7.9. When running the com/sun/jdi tests in a 
>> batch, in samevm mode, this test leave behind a file, NUL, in the 
>> scratch directory that cannot be deleted. This causes the remaining 
>> tests to also fail.
>>
>> Thanks for the review,
>> -Chris.
>>
>> On 08/ 9/11 04:52 PM, Daniel D. Daugherty wrote:
>>> On 8/9/11 8:52 AM, Chris Hegarty wrote:
>>>> Hi Dan, Alan,
>>>>
>>>> This is a trivial change to use /dev/null instead of NUL for Windows
>>>> test with Cygwin.
>>>>
>>>> http://cr.openjdk.java.net/~chegar/7076756/webrev.00/webrev/
>>>>
>>>> Thanks,
>>>> -Chris.
>>>
>>> I'm good with the change, but the bug report itself is less than clear.
>>>
>>> I run these tests all the time on WinXP/Cygwin so I'm guessing
>>> that there is more going on here than meets the eye. I'm going to
>>> guess that the '-samevm' option is the enabler for this failure
>>> mode. The failure mode that you fixed should also affect tests
>>> other than BreakpointWithFullGC.sh so I'm not quite sure why
>>> this issue appears with just that test. Or maybe it does appear
>>> with other tests, but again the bug report isn't clear.
>>>
>>> In any case, thumbs up on the change.
>>>
>>> Dan
>>>