diff options
| author |  Jonathan Nieder <jrnieder@gmail.com> | 2010-02-22 02:46:33 -0600 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2010-02-22 14:19:28 -0800 |
| commit | 8c33b4cf67f47ee46fe0984751fd40c4cf7cf392 (patch) | |
| tree | 6d9e4d65b289b81558de3a57f9adc16a979742d6 /t/t4100/t-apply-3.patch | |
| parent | t7006-pager: if stdout is not a terminal, make a new one (diff) | |
| download | tgif-8c33b4cf67f47ee46fe0984751fd40c4cf7cf392.tar.xz | |
tests: Fix race condition in t7006-pager
Pagers that do not consume their input are dangerous: for example,
$ GIT_PAGER=: git log
$ echo $?
141
$
The only reason these tests were able to work before was that
'git log' would write to the pipe (and not fill it) before the
pager had time to terminate and close the pipe.
Fix it by using a program that consumes its input, namely wc (as
suggested by Johannes).
Reported-by: Johannes Sixt <j.sixt@viscovery.net>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t4100/t-apply-3.patch')
0 files changed, 0 insertions, 0 deletions