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
>  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  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