Giampaolo Rodola' wrote: >>> 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
Tim Golden >> Funnily enough, that's exactly what the patch I've >> put together does. > > Sorry but maybe I misunderstood what you said above. > It seems to me you proposed to add a new "remove_file" function to > test_support while the solution I suggested was modifying the current > test_support.unlink() function in a similar manner you proposed and > have all tests use it wherever it is possible. I wasn't very clear. What I posted earlier was by way of a proof-of-concept and did indeed use a "remove" function. After other people's +1 I have put together a possible patch which does what you're suggesting. I haven't posted it yet and I need to work through the test files checking what the impact would be. I like your second suggestion BTW, altho' I'd probably skip the "if os.name == 'nt'" line and just give it a try in any case. 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
