Tim Golden wrote:
> Giampaolo Rodola' wrote:
>> On 1 Apr, 18:27, "Steven Bethard" <[EMAIL PROTECTED]> wrote:
>>> On Tue, Apr 1, 2008 at 10:20 AM, Facundo Batista
>>>
>>> <[EMAIL PROTECTED]> wrote:
>>>> 2008/4/1, Tim Golden <[EMAIL PROTECTED]>:
>>>> > If this is the thing to do, presumably test_support should
>>>> > grow a "remove_file" which does something of this sort?
>>>> +1 (I was thinking exactly that).
>>> +1 here too. That looks like a great solution. Of course, once it's
>>> in test_support, we need to fix *all* file removals in the test suite.
>>> ;-)
>>>
>>> Steve
>> Why not just modifying test_support.unlink() like this?
>> Surely more convenient than modifying the whole suite.
>>
>>
>> def unlink(filename):
>> try:
>> if os.name == 'nt':
>> os.rename(filename, filename + ".deleted")
>> filename = filename + ".deleted"
>> os.unlink(filename)
>> except OSError:
>> pass
>
> Funnily enough, that's exactly what the patch I've
> put together does. Have to look out, though for
> anywhere which just calls os.unlink rather than
> going through test_support.
Which turns out to be at least 117 instances:
C:\work-in-progress\python26\Lib\test>grep "os.unlink" *.py | wc -l
117
(including the patch I just posted for os.access. Oops).
TJG
_______________________________________________
Python-Dev mailing list
Python-Dev@python.org
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe:
http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com
