Jeff King <[email protected]> writes: > Your revised patch 2 looks good to me. I think you could test it more > reliably by simply adding a larger file, like: > > test-genrandom foo $((128 * 1024 + 1)) >big && > echo 'big filter=epipe' >.gitattributes && > git config filter.epipe.clean true && > git add big > > The worst case if you get the size of the pipe buffer too small is that > the test will erroneously pass, but that is OK. As long as one person > has a reasonable-sized buffer, they will complain to the list > eventually. :)
Yeah, I like it. It was lazy of me not to add a new test. Thanks. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
