~fullermd/brz/silence-git-apply-test

  • Committer: Matthew Fuller
  • Date: 2019-03-24 20:47:35 UTC
  • Revision ID: fullermd@over-yonder.net-20190324204735-6fhk08s8invbc3gz
Silence output from patch(1) in git-apply test.

The test suite stubs in stdout/err wrappers for the python code, but
doesn't mess with the underlying file descriptors, so spawned off
processes still get the terminal.  cmd_git_apply._apply_patch() uses
subprocess to spawn off the system patch(1) to do the application, and
patch talks a lot about what it's doing.  While patch has a -s arg to
quiet it down, we probably want it speaking a lot in normal use so the
user can see what's going on, so just switch stdout to /dev/null
around the run_bzr call in the test instead.

An argument could be made that the test suite as a whole should be
better about moving the fd's so subprocesses can't hit the terminal
directly, but for the moment, this seems to be the only test where it
matters.
Filename Latest Rev Last Changed Committer Comment Size
..
apport 4797.33.16 14 years ago Robert Collins Update bugfix for 528114 to trunk. Diff
breezy 6622.1.34 6 years ago Jelmer Vernooij Rename brzlib => breezy. Diff
contrib 1185.1.29 18 years ago Robert Collins merge merge tweaks from aaron, which includes late Diff
doc 1185.1.29 18 years ago Robert Collins merge merge tweaks from aaron, which includes late Diff
man1 2425.1.1 17 years ago Robert Collins ``make docs`` now creates a man page at ``man1/bzr Diff
po 5830.2.1 12 years ago INADA Naoki Add update-pot command to Makefile and tools/bzrge Diff
tools 1185.1.29 18 years ago Robert Collins merge merge tweaks from aaron, which includes late Diff
.bzrignore 7290.12.2 5 years ago Vincent Ladeuil Release 3.0.0 1.3 KB Diff Download File
.coveragerc 6954.1.4 5 years ago Jelmer Vernooij Specify source in .coveragerc. 92 bytes Diff Download File
.mailmap 6883.23.19 5 years ago Jelmer Vernooij Move .mailmap to root. 299 bytes Diff Download File
.rsyncexclude 1185.33.36 18 years ago Martin Pool Exclude more files from dumb-rsync upload 203 bytes Diff Download File
.testr.conf 6973.12.6 5 years ago Jelmer Vernooij Merge test-run support. 150 bytes Diff Download File
.travis.yml 7259 5 years ago Breezy landing bot Ignore UserWarning on travis. Merged from https:/ 826 bytes Diff Download File
BRANCH.TODO 5340.8.6 13 years ago Marius Kruger * Update documentation * add NEWS 150 bytes Diff Download File
File brz 7294.1.2 5 years ago Vincent Ladeuil Open trunk again as 3.1.0dev1 3.6 KB Diff Download File
byov.conf 7289.2.3 5 years ago Jelmer Vernooij Install cython from pip in byov. 3.2 KB Diff Download File
COPYING.txt 6613.2.1 8 years ago Richard Wilbur Update COPYING.txt from the original http://www.gn 17.6 KB Diff Download File
INSTALL 6794 6 years ago Jelmer Vernooij Merge lp:~jelmer/brz/install. 1.8 KB Diff Download File
Makefile 5255.6.5 5 years ago Jelmer Vernooij Merge trunk. 11.5 KB Diff Download File
MANIFEST.in 6929.2.2 6 years ago Jelmer Vernooij README -> README.rst in more places. 854 bytes Diff Download File
NEWS 6690.6.5 6 years ago Jelmer Vernooij Merge trunk, address review comments. 141 bytes Diff Download File
profile_imports.py 7195.5.1 5 years ago Martin Fix remaining whitespace lint in codebase Enables 6.3 KB Diff Download File
README.rst 7268.5.1 5 years ago Jelmer Vernooij Add a mention of Python 3 support in README.rst. 2.4 KB Diff Download File
README_BDIST_RPM 4634.125.1 14 years ago Joe Julian Added MANIFEST.in to allow bdist_rpm to have all t 397 bytes Diff Download File
setup.cfg 7205 5 years ago Breezy landing bot Fix remaining whitespace lint in codebase Merged 540 bytes Diff Download File
File setup.py 7290.1.14 5 years ago Breezy landing bot Don't use cython if it's too old. Merged from htt 29 KB Diff Download File
TODO 6858.1.1 6 years ago Jelmer Vernooij Update TODO to reference Breezy, not Bazaar. 77 bytes Diff Download File