On Mon, Feb 15, 2016 at 09:36:23PM +0000, Ramsay Jones wrote: > > +test_expect_success '--show-origin stdin' ' > > + cat >expect <<-\EOF && > > + stdin: user.custom=true > > So, as with the previous patch, I think this should be: > file:<stdin> user.custom=true
That's ambiguous with a file named "<stdin>", which was the point of having the two separate prefixes in the first place. I think in practice we _could_ get by with an ambiguous output (it's not like "<stdin>" is a common filename), but that was discussed earlier in the thread, and Lars decided to go for something unambiguous. That doesn't necessarily have to bleed over into the error messages, though (which could continue to use "<stdin>" if we want to put in a little extra code to covering the cases separately. -Peff -- 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
