Issue1598

The bug can be reproduced by running the code below on Windows:

-----8<-----------8<---------
from xml.etree import ElementTree as ET
import os

path = 'temp.xml'
f = open(path, 'w')
f.write('<whatever/>')
f.close()

ET.parse(path)
os.remove(path)
-----8<-----------8<---------

The result of execution is this error:

Traceback (most recent call last):
  File "rmbug.py", line 10, in <module>
    os.remove(path)
  File "C:\jython2.5.1\Lib\os.py", line 342, in remove
    raise OSError(0, "couldn't delete file", path)
OSError: [Errno 0] couldn't delete file: 'temp.xml'

The reason for the bug is the ElementTree.parse doesn't close the source file it opens. This bug doesn't appear with CPython because there the source file is immediately garbage collected and thus automatically closed when the parse method is exited. On Unixes the example in the original description passes also with Jython, apparently because there the OS doesn't lock the opened file similarly as on Windows.

I tested that adding `source.close()` at the end of the parse method was enough to make the original example pass. That's probably not the right way to fix this, though, as the source should be closed only if it's opened inside parse. I don't have Jython development environment set up and cannot create patch now, but I can do that later if there is a need.

IMHO code in Python stdlib shouldn't rely on CPython gc behavior and the fix should thus be done there. Should I open a separate issue there?

dupe of #1479. there's already a CPython ticket

I seem to be good in finding already reported bugs. This time I actually search the tracker before submitting but apparently I searched only the open issues. Anyway, great that this is fixed and hope 2.5.2 is out soon!