path: root/t/
diff options
authorKirill Smelkov <>2010-12-18 14:54:12 (GMT)
committerJunio C Hamano <>2010-12-20 02:41:32 (GMT)
commit9ec09b0495f5e672223025f5e681371034da2538 (patch)
treefa3031a4d139c21da6e416e2280f47d97018bd11 /t/
parentb1b14ec3f14edfc5f76643aaaac6548fc94fb47e (diff)
fill_textconv(): Don't get/put cache if sha1 is not valid
When blaming files in the working tree, the filespec is marked with !sha1_valid, as we have not given the contents an object name yet. The function to cache textconv results (keyed on the object name), however, didn't check this condition, and ended up on storing the cached result under a random object name. Cc: Axel Bonnet <> Cc: Clément Poulain <> Cc: Diane Gasselin <> Cc: Jeff King <> Signed-off-by: Kirill Smelkov <> Signed-off-by: Junio C Hamano <>
Diffstat (limited to 't/')
1 files changed, 1 insertions, 2 deletions
diff --git a/t/ b/t/
index fe90541..ea64cd8 100755
--- a/t/
+++ b/t/
@@ -81,8 +81,7 @@ cat >expected_one <<EOF
(Number2 2010-01-01 20:00:00 +0000 1) converted: test 1 version 2
-# one.bin is blamed as 'Not Committed yet'
-test_expect_failure 'blame --textconv works with textconvcache' '
+test_expect_success 'blame --textconv works with textconvcache' '
git blame --textconv two.bin >blame &&
find_blame <blame >result &&
test_cmp expected result &&