On Mon, Jun 25, 2012 at 6:32 PM, Nguyen Thai Ngoc Duy <pclouds@xxxxxxxxx> wrote: >>> --- a/t/t0006-date.sh >>> +++ b/t/t0006-date.sh >>> @@ -9,7 +9,7 @@ TEST_DATE_NOW=1251660000; export TEST_DATE_NOW >>> check_show() { >>> t=$(($TEST_DATE_NOW - $1)) >>> echo "$t -> $2" >expect >>> - test_expect_${3:-success} "relative date ($2)" " >>> + test_expect_${3:-success} C_LOCALE_OUTPUT "relative date ($2)" " >>> test-date show $t >actual && >>> test_cmp expect actual >>> " >> >> Could use test_i18ncmp so we catch if test-date crashes, but anyway, >> yeah, we can't expect to be able to meaningfully test date formatting >> in another language. > > This reminds me of an inkscape crash I encountered long ago because of > a mistake in Vietnamese translation. Too bad we can't test all locales > for similar bugs (or we would need to i18n-ize expected results in the > test suite too) What if we allow to run the test suite in non-C locales and skip the tests that require C_LOCALE_OUTPUT? In non-C locales, we won't set C_LOCALE_OUTPUT prerequisite. This allows to test a localized git with ~90% of the test suite. Would it work? -- Duy -- 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