Alexey Shumkin <alex.crez...@gmail.com> writes:

> @@ -19,7 +23,8 @@ add_file () {
>                       echo "$name" >"$name" &&
>                       git add "$name" &&
>                       test_tick &&
> -                     git commit -m "Add $name" || exit
> +                     msg_added_iso88595=$(echo "Add $name ($added $name)" | 
> iconv -f utf-8 -t iso88595) &&
> +                     git -c 'i18n.commitEncoding=iso88595' commit -m 
> "$msg_added_iso88595"

Hmph.  Do we know 8859-5 is available or do these need to be
protected with prereq?

Can these tests be done with 8859-1 i.e. something we already depend
on, by changing that $added message to latin-1, or is there something
very Russian specific breakage we want to test here?

If the former, please redo this entire patch (not just t4041) with
8859-1.

If there is some Russian specific breakage you cannot demonstrate
with 8859-1, then please explain it in the log message.

> diff --git a/t/t6006-rev-list-format.sh b/t/t6006-rev-list-format.sh
> index d32e65e..36e4cc0 100755
> --- a/t/t6006-rev-list-format.sh
> +++ b/t/t6006-rev-list-format.sh
> ...
> -# usage: test_format name format_string <expected_output
> +# usage: test_format [failure] name format_string <expected_output
>  test_format () {
> +     local must_fail=0

This breaks tests for non-bash users.  "local" is not even in POSIX.
--
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

Reply via email to