Philip Martin <philip.mar...@wandisco.com> writes: > Daniel Shahaf <d...@daniel.shahaf.name> writes: > >>> > handle. Solutions include: >>> > >>> > - adding an svn_repos_close API >>> > - clearing or destroying the pool passed to svn_repos_create >>> > - having the test create repositories at different locations >>> >>> I have implemented the last solution and attached is the patch for the >>> same. >> >> I think Philip was just enumerating all possible solutions. I think >> making the test suite avoid the problem is the wrong solution --- it >> would be better to cause those dangling handles to get closed at the >> appropriate time (e.g., when there are no more references to the Python >> repos object). > > It may be as simple as > > Index: ../src/subversion/bindings/ctypes-python/test/wc.py > =================================================================== > --- ../src/subversion/bindings/ctypes-python/test/wc.py (revision 1031593) > +++ ../src/subversion/bindings/ctypes-python/test/wc.py (working copy) > @@ -72,6 +72,7 @@ > if os.path.exists(wc_location): > svn_io_remove_dir(wc_location, pool) > if os.path.exists(repos_location): > + self.repos = None > svn_repos_delete(repos_location, pool) > self.wc = None >
This did not solve the problem. I still get the disk I/O error. > > I don't know how to run the ctypes tests. make check-ctypes-python Thanks and Regards Noorul
