On Fri, Jul 05, 2013 at 12:04:34AM -0700, Junio C Hamano wrote: > Alexey Shumkin <alex.crezoff@xxxxxxxxx> writes: > > > +test_format complex-body %b <<EOF > > +commit $head3 > > +This commit message is much longer than the others, > > +and it will be encoded in iso8859-1. We should therefore > > +include an iso8859 character: ¡bueno! > > This is not such a good idea, as the resulting file will be in mixed > encoding (it already has a line with non-ascii that is in UTF-8), > and many editors would not like such a file. I agree, there was issues when I edited that file > > Perhaps we should update test_format so that we can feed a quoted > input, e.g. > > +include an iso8859-1 character: <A1>bueno! > > or something? We could use this file whole in UTF-8 but just make a conversion of expected output as it's done a few lines above with a commit message (stored to a file 'commit-msg' before the test 'setup complex body'). +iconv -f utf-8 -t iso8859-1 > commit-msg <<EOF +Test printing of complex bodies We can even use content of that file (as far as it's kept untouched between that tests) > > > +commit $head2 > > +commit $head1 > > +EOF > > + > > +# Git uses i18n.commitEncoding if no i18n.logOutputEncoding set > > +# so unset i18n.commitEncoding to test encoding conversion > > +git config --unset i18n.commitEncoding > > + > > +test_format complex-subject-commitencoding-unset %s <<EOF > > +commit $head3 > > +Test printing of complex bodies > > +commit $head2 > > $changed > > commit $head1 > > $added > > EOF > > > > -test_format complex-body %b <<EOF > > +test_format complex-body-commitencoding-unset %b <<EOF > > commit $head3 > > This commit message is much longer than the others, > > and it will be encoded in iso8859-1. We should therefore -- Alexey Shumkin -- 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