Jeff King venit, vidit, dixit 20.04.2013 06:04: > On Fri, Apr 19, 2013 at 06:44:44PM +0200, Michael J Gruber wrote: > >> "git show <commit>" obeys the textconc setting while "git show <blob>" >> does not. Demonstrate this in the test. > > s/textconc/textconv
Thanks, plus s/obey/honor/ >> diff --git a/t/t4030-diff-textconv.sh b/t/t4030-diff-textconv.sh >> index 53ec330..f314ced 100755 >> --- a/t/t4030-diff-textconv.sh >> +++ b/t/t4030-diff-textconv.sh >> @@ -58,6 +58,12 @@ test_expect_success 'diff produces text' ' >> test_cmp expect.text actual >> ' >> >> +test_expect_success 'show commit produces text' ' >> + git show HEAD >diff && >> + find_diff <diff >actual && >> + test_cmp expect.text actual >> +' > > Makes sense. > >> +test_expect_success 'show blob produces binary' ' >> + git show HEAD:file >actual && >> + printf "\\0\\n\\1\\n" >expect && >> + test_cmp expect actual >> +' > > I think this is probably the right thing. I can see instances where one > would want the converted contents, but we have "cat-file --textconv" for > that. > By that you mean that this behavior is to stay as is? My reasoning is twofold: - consistency between "git show commit" and "git show blob" - "git show" is a user facing command, and as such should produce output consumable by humans; whereas "git cat-file" is plumbing and should produce raw data unless told otherwise (-p, --textconv). Michael -- 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
