> -----Original Message----- > From: [email protected] [mailto:[email protected]] > Sent: dinsdag 6 april 2010 16:20 > To: [email protected] > Subject: svn commit: r931162 - in /subversion/trunk: ./ > subversion/libsvn_diff/parse-diff.c subversion/tests/cmdline/patch_tests.py > > Author: stsp > Date: Tue Apr 6 14:20:21 2010 > New Revision: 931162 > > URL: http://svn.apache.org/viewvc?rev=931162&view=rev > Log: > Cherry-pick r929288 and r931140 from the svn-patch-improvements branch > to trunk, fixing the problem reported in: > > Date: Wed, 3 Mar 2010 15:52:59 +0100 > From: Stefan Sperling > To: [email protected] > Subject: svn patch fails on this diff > Message-ID: <[email protected]> > http://svn.haxx.se/dev/archive-2010-03/0097.shtml > http://svn.haxx.se/dev/archive-2010-03/0098.shtml (attachment) >
> Modified: subversion/trunk/subversion/tests/cmdline/patch_tests.py > URL: > http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/ > patch_tests.py?rev=931162&r1=931161&r2=931162&view=diff > ========================================================== > ==================== > --- subversion/trunk/subversion/tests/cmdline/patch_tests.py (original) > +++ subversion/trunk/subversion/tests/cmdline/patch_tests.py Tue Apr 6 > 14:20:21 2010 > @@ -841,6 +841,92 @@ def patch_strip1(sbox): > 1, # dry-run > '-p1') > > +def patch_no_index_line(sbox): > + "patch with no index lines" > + > + sbox.build() > + wc_dir = sbox.wc_dir > + > + patch_file_path = > tempfile.mkstemp(dir=os.path.abspath(svntest.main.temp_dir))[1] This line reintroduces the issue I fixed in the other tests in r930990. Stefan: Answering your question: It seems the handle must be closed before leaving the function. (I don't mind where, as long as it fixes the errors) Greg: You just fixed several other functions to use open(...).read(), without closing the handle. This doesn't cause the same error, so this file is somehow closed without the explicit close required above. Do you have any idea why mkstemp() does need the explicit close? (Python bug?) Other question: Do we really need a mkstemp call here? I assumed we have a per test temporary directory where we can place any named file we want? Bert
