Re: [PATCH 5/7] Fix tests under GETTEXT_POISON on pack-object
On Tue, Aug 21, 2012 at 12:17 PM, Jonathan Nieder jrnie...@gmail.com wrote: --- a/t/t5530-upload-pack-error.sh +++ b/t/t5530-upload-pack-error.sh @@ -35,7 +35,7 @@ test_expect_success 'upload-pack fails due to error in pack-objects packing' ' printf 0032want %s\n0009done\n \ $(git rev-parse HEAD) input test_must_fail git upload-pack . input /dev/null 2output.err - grep unable to read output.err + test_i18ngrep unable to read output.err grep pack-objects died output.err Wouldn't it make sense to change the second grep of output intended for humans to test_i18ngrep while at it? This comes from error(git upload-pack: git-pack-objects died with error.) in unpack-trees.c, which is not i18n-ized yet. There's another test in t5530 that does the same grep. I think we should leave it as is until we mark the string for translation, then gettext poison will spot it (verified) and we can fix it. -- Duy -- 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
Re: [PATCH 5/7] Fix tests under GETTEXT_POISON on pack-object
Nguyen Thai Ngoc Duy wrote: On Tue, Aug 21, 2012 at 12:17 PM, Jonathan Nieder jrnie...@gmail.com wrote: @@ -35,7 +35,7 @@ test_expect_success 'upload-pack fails due to error in pack-objects packing' ' printf 0032want %s\n0009done\n \ $(git rev-parse HEAD) input test_must_fail git upload-pack . input /dev/null 2output.err - grep unable to read output.err + test_i18ngrep unable to read output.err grep pack-objects died output.err Wouldn't it make sense to change the second grep of output intended for humans to test_i18ngrep while at it? This comes from error(git upload-pack: git-pack-objects died with error.) in unpack-trees.c, which is not i18n-ized yet. There's another test in t5530 that does the same grep. I think we should leave it as is until we mark the string for translation, then gettext poison will spot it (verified) and we can fix it. I don't understand the distinction you're making. Isn't the message intended for humans, and wouldn't changing that one line to test_i18ngrep now save trouble later? Tests are meant to check git's intended behavior, not to exactly match its current behavior. Jonathan -- 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
Re: [PATCH 5/7] Fix tests under GETTEXT_POISON on pack-object
Jonathan Nieder jrnie...@gmail.com writes: Nguyen Thai Ngoc Duy wrote: On Tue, Aug 21, 2012 at 12:17 PM, Jonathan Nieder jrnie...@gmail.com wrote: @@ -35,7 +35,7 @@ test_expect_success 'upload-pack fails due to error in pack-objects packing' ' printf 0032want %s\n0009done\n \ $(git rev-parse HEAD) input test_must_fail git upload-pack . input /dev/null 2output.err - grep unable to read output.err + test_i18ngrep unable to read output.err grep pack-objects died output.err Wouldn't it make sense to change the second grep of output intended for humans to test_i18ngrep while at it? This comes from error(git upload-pack: git-pack-objects died with error.) in unpack-trees.c, which is not i18n-ized yet. There's another test in t5530 that does the same grep. I think we should leave it as is until we mark the string for translation, then gettext poison will spot it (verified) and we can fix it. I don't understand the distinction you're making. Isn't the message intended for humans, and wouldn't changing that one line to test_i18ngrep now save trouble later? Tests are meant to check git's intended behavior, not to exactly match its current behavior. Correct; your suggestion is good. -- 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
Re: [PATCH 5/7] Fix tests under GETTEXT_POISON on pack-object
Hi, Nguyễn Thái Ngọc Duy wrote: From: Jiang Xin worldhello@gmail.com Use i18n-specific test functions in test scripts for pack-object. Thanks for resending, and sorry I haven't made time to polish the translation-based poison implementation you sent before (which seemed very useful and pleasant to work with). [...] --- a/t/t5530-upload-pack-error.sh +++ b/t/t5530-upload-pack-error.sh @@ -35,7 +35,7 @@ test_expect_success 'upload-pack fails due to error in pack-objects packing' ' printf 0032want %s\n0009done\n \ $(git rev-parse HEAD) input test_must_fail git upload-pack . input /dev/null 2output.err - grep unable to read output.err + test_i18ngrep unable to read output.err grep pack-objects died output.err Wouldn't it make sense to change the second grep of output intended for humans to test_i18ngrep while at it? With or without that change, this and the rest of the series looks good. Hope that helps, Jonathan -- 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