On Wed, Oct 24, 2012 at 08:33:15AM +0200, Johannes Sixt wrote: > Am 10/24/2012 0:35, schrieb Jeff King: > > -test_expect_success 'non-match value' \ > > - 'test wow = $(git config --get nextsection.nonewline !for)' > > +test_expect_success 'non-match value' ' > > + test wow = $(git config --get nextsection.nonewline !for) > > +' > > Here's a case you forgot to update to test_cmp. Thanks. I noticed I left quite a few of those in (the other changes I did mechanically, but I only fixed up the "test" ones in nearby spots). > > +test_expect_success 'get-regexp variable with no value' ' > > + git config --get-regexp novalue > output && > > + test_cmp expect output' > > And while you are here, you might want to remove this extra space. ;) > > Otherwise, looks fine. Thanks, I'll fix up both. -Peff -- 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