Re: sh -x -i -v with continuous integration, was Re: [PATCH 1/4] t7601: test for pull.ff=true overrides merge.ff=false

[Johannes Schindelin]

Hi Paul,

On 2015-05-16 14:33, Paul Tan wrote:

 On Fri, May 15, 2015 at 12:53 AM, Johannes Schindelin
 johannes.schinde...@gmx.de wrote:
 Asketh and ye shall be given: without running the tests in parallel, our 
 Jenkins would take *even longer* than the three hours per test suite run 
 (which is really painful, still, by the way). And as you know, running the 
 tests with -v -x is awfully useless if you run the test suite in parallel.
 
 Would the --tee option work for this case?
 
 --tee::
 In addition to printing the test output to the terminal,
 write it to files named 't/test-results/$TEST_NAME.out'.
 As the names depend on the tests' file names, it is safe to
 run the tests with this option in parallel.

Thanks for pointing this out to me. This is such a useful option.

Now, who made that patch... Oh, wow, oops [*1*]. Time for a coffee, I guess.

Thank you!
Dscho

Footnote *1*: 
https://github.com/git/git/commit/44138559e8b7c89768a2450220b831847059311c
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: sh -x -i -v with continuous integration, was Re: [PATCH 1/4] t7601: test for pull.ff=true overrides merge.ff=false

[Junio C Hamano]

Jeff King p...@peff.net writes:

 What do you want to do with the verbose calls I have been sprinkling
 through the test suite (and the function itself)? Leave them or remove
 them? A grep for verbose  (with the trailing space) shows some
 examples.

That was one of the things I wanted to do next, as I suspected that
the small number of existing ones might be mostly obviously good
ones (I never said any 'verbose test' is automatically bad; it is
just 'verbose test $a = $b' that hides what $a and $b were are
as useful as without 'verbose'), but I didn't offhand think of
examples of obviously good ones.



--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html