To get this branch, use:
bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
- Committer: John Arbash Meinel
- Date: 2011-04-20 14:27:19 UTC
- Revision ID: john@arbash-meinel.com-20110420142719-advs1k5vztqzbrgv
Fix bug #767177. Be more agressive with file.close() calls.
Our test suite gets a number of thread leaks and failures because it happens to get async
SFTPFile.close() calls. (if an SFTPFile closes due to __del__ it is done as an async request,
while if you call SFTPFile.close() it is done as a synchronous request.)
We have a couple other cases, probably. Namely SFTPTransport.get() also does an async
prefetch of the content, so if you don't .read() you'll also leak threads that think they
are doing work that you want.
The biggest change here, though, is using a try/finally in a generator, which is not
python2.4 compatible.
Our test suite gets a number of thread leaks and failures because it happens to get async
SFTPFile.close() calls. (if an SFTPFile closes due to __del__ it is done as an async request,
while if you call SFTPFile.close() it is done as a synchronous request.)
We have a couple other cases, probably. Namely SFTPTransport.get() also does an async
prefetch of the content, so if you don't .read() you'll also leak threads that think they
are doing work that you want.
The biggest change here, though, is using a try/finally in a generator, which is not
python2.4 compatible.
| Filename | Latest Rev | Last Changed | Committer | Comment | Size | ||
|---|---|---|---|---|---|---|---|
 .. |
|||||||
apport
|
4797.33.16 | 15 years ago | Robert Collins | Update bugfix for 528114 to trunk. |
|
||
|
bzrlib
|
1185.1.29 | 20 years ago | Robert Collins | merge merge tweaks from aaron, which includes late |
|
||
|
contrib
|
1185.1.29 | 20 years ago | Robert Collins | merge merge tweaks from aaron, which includes late |
|
||
|
doc
|
1185.1.29 | 20 years ago | Robert Collins | merge merge tweaks from aaron, which includes late |
|
||
|
man1
|
2425.1.1 | 18 years ago | Robert Collins | ``make docs`` now creates a man page at ``man1/bzr |
|
||
|
tools
|
1185.1.29 | 20 years ago | Robert Collins | merge merge tweaks from aaron, which includes late |
|
||
.bzrignore |
5462.5.3 | 15 years ago | Andrew Bennetts | Keep docs-plain working. | 1.2 KB |
|
|
|
.rsyncexclude |
1185.33.36 | 20 years ago | Martin Pool | Exclude more files from dumb-rsync upload | 203 bytes |
|
|
|
.testr.conf |
5574.2.2 | 15 years ago | Jelmer Vernooij | Don't use no-plugins. | 126 bytes |
|
|
|
BRANCH.TODO |
5340.8.6 | 15 years ago | Marius Kruger | * Update documentation * add NEWS | 150 bytes |
|
|
bzr |
5713.1.1 | 14 years ago | Vincent Ladeuil | Merge 2.3 into trunk including fix for bug #437003 | 5.2 KB |
|
|
|
bzr.ico |
3688.3.3 | 17 years ago | John Arbash Meinel | An updated transparent icon for bzr. | 12.7 KB |
|
|
|
COPYING.txt |
1861.2.9 | 19 years ago | Alexander Belchenko | rename gpl.txt => COPYING.txt | 17.5 KB |
|
|
|
INSTALL |
5278.1.5 | 15 years ago | Martin Pool | Correct more sloppy use of the term 'Linux' | 1.4 KB |
|
|
|
Makefile |
5793.3.1 | 14 years ago | Martin Pool | Use -Wignore::ImportWarning rather than catching i | 15.1 KB |
|
|
|
MANIFEST.in |
4634.125.2 | 16 years ago | John Arbash Meinel | Update the MANIFEST.in file. | 118 bytes |
|
|
|
NEWS |
5462.5.1 | 15 years ago | Andrew Bennetts | Split NEWS into per-release doc/en/release-notes/b | 141 bytes |
|
|
|
profile_imports.py |
5027 | 15 years ago | Canonical.com Patch Queue Manager | (mbp) add import-tariff tests | 6.1 KB |
|
|
|
README |
5447.2.1 | 15 years ago | Vincent Ladeuil | Fix some typos and propose a release planning. | 2.2 KB |
|
|
|
README_BDIST_RPM |
4634.125.1 | 16 years ago | Joe Julian | Added MANIFEST.in to allow bdist_rpm to have all t | 397 bytes |
|
|
|
setup.py |
5555.3.1 | 15 years ago | IWATA Hidetaka | add icon resources tbzrcommand(w) | 28.5 KB |
|
|
|
TODO |
4634.146.5 | 15 years ago | Danny van Heumen | Reverted now-useless TODO list. | 115 bytes |
|
|