diff options
| author | Junio C Hamano <gitster@pobox.com> | 2019-02-09 19:25:26 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2019-02-11 21:31:19 +0100 |
| commit | 11f470aee7ccd43bc2be159e69e121c35a72f91d (patch) | |
| tree | 076e1ca43d4b14a12ee085ffe69817180814d557 /t | |
| parent | Seventh batch for 2.21 (diff) | |
| download | git-11f470aee7ccd43bc2be159e69e121c35a72f91d.tar.xz git-11f470aee7ccd43bc2be159e69e121c35a72f91d.zip | |
test: caution on our version of 'yes'
During a review of a patch, we noticed that we use our own imitation
of 'yes' with the limit of 99 lines. It is very tempting to lift this
arbitrary limit, but the limit is there for a reason.
Add an in-code comment to prevent future developers from wasting
their time.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't')
| -rw-r--r-- | t/README | 9 | ||||
| -rw-r--r-- | t/test-lib.sh | 6 |
2 files changed, 14 insertions, 1 deletions
@@ -927,6 +927,15 @@ library for your script to use. test_oid_init or test_oid_cache. Providing an unknown key is an error. + - yes [<string>] + + This is often seen in modern UNIX but some platforms lack it, so + the test harness overrides the platform implementation with a + more limited one. Use this only when feeding a handful lines of + output to the downstream---unlike the real version, it generates + only up to 99 lines. + + Prerequisites ------------- diff --git a/t/test-lib.sh b/t/test-lib.sh index 42b1a0aa7f..541a37f4c0 100644 --- a/t/test-lib.sh +++ b/t/test-lib.sh @@ -1313,7 +1313,11 @@ then fi fi -# Provide an implementation of the 'yes' utility +# Provide an implementation of the 'yes' utility; the upper bound +# limit is there to help Windows that cannot stop this loop from +# wasting cycles when the downstream stops reading, so do not be +# tempted to turn it into an infinite loop. cf. 6129c930 ("test-lib: +# limit the output of the yes utility", 2016-02-02) yes () { if test $# = 0 then |