Thomas Rast <trast@xxxxxxxxxxx> writes: >> + grep -v " " error > > Umm, doesn't that only test that _some_ line in the error does not > contain a tab? Indeed. It does work as the error message is just a one-liner, but let's be robust anyway. > Whereas you need to test that _no_ line contains <TAB>emp, or some > such. Perhaps as > > ! grep -v " emp" error Err, then the -v should be remove. Also, I'll use test_expect_code 1 instead of ! to catch other grep failures, just in case. Thanks, new patch comming. -- Matthieu Moy http://www-verimag.imag.fr/~moy/ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html