i18n: git-commit "enter the commit message" message
Gettextize the "# Please enter the commit message for your changes." message. Several tests in t7500-commit.sh and t7502-commit.sh assume that this message starts with a newline. Change the tests to to skip under GETTEXT_POISON=YesPlease. These fail under GETTEXT_POISON=YesPlease because the poison editor message doesn't do the right thing with comments and line breaks, so these messages will be incorrectly broken up across lines. This test should not be skipped under a hypothetical future testing mode that tests Git under language locales, since those messages should all start with a newline like the original. Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
7f5673d703
commit
0b430a17ae
@ -705,18 +705,18 @@ static int prepare_to_commit(const char *index_file, const char *prefix,
|
|||||||
|
|
||||||
fprintf(s->fp, "\n");
|
fprintf(s->fp, "\n");
|
||||||
status_printf(s, GIT_COLOR_NORMAL,
|
status_printf(s, GIT_COLOR_NORMAL,
|
||||||
"Please enter the commit message for your changes.");
|
_("Please enter the commit message for your changes."));
|
||||||
if (cleanup_mode == CLEANUP_ALL)
|
if (cleanup_mode == CLEANUP_ALL)
|
||||||
status_printf_more(s, GIT_COLOR_NORMAL,
|
status_printf_more(s, GIT_COLOR_NORMAL,
|
||||||
" Lines starting\n"
|
_(" Lines starting\n"
|
||||||
"with '#' will be ignored, and an empty"
|
"with '#' will be ignored, and an empty"
|
||||||
" message aborts the commit.\n");
|
" message aborts the commit.\n"));
|
||||||
else /* CLEANUP_SPACE, that is. */
|
else /* CLEANUP_SPACE, that is. */
|
||||||
status_printf_more(s, GIT_COLOR_NORMAL,
|
status_printf_more(s, GIT_COLOR_NORMAL,
|
||||||
" Lines starting\n"
|
_(" Lines starting\n"
|
||||||
"with '#' will be kept; you may remove them"
|
"with '#' will be kept; you may remove them"
|
||||||
" yourself if you want to.\n"
|
" yourself if you want to.\n"
|
||||||
"An empty message aborts the commit.\n");
|
"An empty message aborts the commit.\n"));
|
||||||
if (only_include_assumed)
|
if (only_include_assumed)
|
||||||
status_printf_ln(s, GIT_COLOR_NORMAL,
|
status_printf_ln(s, GIT_COLOR_NORMAL,
|
||||||
"%s", only_include_assumed);
|
"%s", only_include_assumed);
|
||||||
|
@ -64,7 +64,7 @@ test_expect_success 'adding comments to a template should not commit' '
|
|||||||
)
|
)
|
||||||
'
|
'
|
||||||
|
|
||||||
test_expect_success 'adding real content to a template should commit' '
|
test_expect_success C_LOCALE_OUTPUT 'adding real content to a template should commit' '
|
||||||
(
|
(
|
||||||
test_set_editor "$TEST_DIRECTORY"/t7500/add-content &&
|
test_set_editor "$TEST_DIRECTORY"/t7500/add-content &&
|
||||||
git commit --template "$TEMPLATE"
|
git commit --template "$TEMPLATE"
|
||||||
@ -72,7 +72,7 @@ test_expect_success 'adding real content to a template should commit' '
|
|||||||
commit_msg_is "template linecommit message"
|
commit_msg_is "template linecommit message"
|
||||||
'
|
'
|
||||||
|
|
||||||
test_expect_success '-t option should be short for --template' '
|
test_expect_success C_LOCALE_OUTPUT '-t option should be short for --template' '
|
||||||
echo "short template" > "$TEMPLATE" &&
|
echo "short template" > "$TEMPLATE" &&
|
||||||
echo "new content" >> foo &&
|
echo "new content" >> foo &&
|
||||||
git add foo &&
|
git add foo &&
|
||||||
@ -83,7 +83,7 @@ test_expect_success '-t option should be short for --template' '
|
|||||||
commit_msg_is "short templatecommit message"
|
commit_msg_is "short templatecommit message"
|
||||||
'
|
'
|
||||||
|
|
||||||
test_expect_success 'config-specified template should commit' '
|
test_expect_success C_LOCALE_OUTPUT 'config-specified template should commit' '
|
||||||
echo "new template" > "$TEMPLATE" &&
|
echo "new template" > "$TEMPLATE" &&
|
||||||
git config commit.template "$TEMPLATE" &&
|
git config commit.template "$TEMPLATE" &&
|
||||||
echo "more content" >> foo &&
|
echo "more content" >> foo &&
|
||||||
@ -282,7 +282,7 @@ test_expect_success 'commit --squash works with -c for same commit' '
|
|||||||
commit_msg_is "squash! edited commit"
|
commit_msg_is "squash! edited commit"
|
||||||
'
|
'
|
||||||
|
|
||||||
test_expect_success 'commit --squash works with editor' '
|
test_expect_success C_LOCALE_OUTPUT 'commit --squash works with editor' '
|
||||||
commit_for_rebase_autosquash_setup &&
|
commit_for_rebase_autosquash_setup &&
|
||||||
test_set_editor "$TEST_DIRECTORY"/t7500/add-content &&
|
test_set_editor "$TEST_DIRECTORY"/t7500/add-content &&
|
||||||
git commit --squash HEAD~1 &&
|
git commit --squash HEAD~1 &&
|
||||||
|
@ -221,19 +221,21 @@ test_expect_success 'cleanup commit messages (strip,-F)' '
|
|||||||
|
|
||||||
'
|
'
|
||||||
|
|
||||||
echo "sample
|
|
||||||
|
|
||||||
# Please enter the commit message for your changes. Lines starting
|
|
||||||
# with '#' will be ignored, and an empty message aborts the commit." >expect
|
|
||||||
|
|
||||||
test_expect_success 'cleanup commit messages (strip,-F,-e)' '
|
test_expect_success 'cleanup commit messages (strip,-F,-e)' '
|
||||||
|
|
||||||
echo >>negative &&
|
echo >>negative &&
|
||||||
{ echo;echo sample;echo; } >text &&
|
{ echo;echo sample;echo; } >text &&
|
||||||
git commit -e -F text -a &&
|
git commit -e -F text -a &&
|
||||||
head -n 4 .git/COMMIT_EDITMSG >actual &&
|
head -n 4 .git/COMMIT_EDITMSG >actual
|
||||||
test_cmp expect actual
|
'
|
||||||
|
|
||||||
|
echo "sample
|
||||||
|
|
||||||
|
# Please enter the commit message for your changes. Lines starting
|
||||||
|
# with '#' will be ignored, and an empty message aborts the commit." >expect
|
||||||
|
|
||||||
|
test_expect_success C_LOCALE_OUTPUT 'cleanup commit messages (strip,-F,-e): output' '
|
||||||
|
test_cmp expect actual
|
||||||
'
|
'
|
||||||
|
|
||||||
echo "#
|
echo "#
|
||||||
@ -244,7 +246,10 @@ test_expect_success C_LOCALE_OUTPUT 'author different from committer' '
|
|||||||
|
|
||||||
echo >>negative &&
|
echo >>negative &&
|
||||||
git commit -e -m "sample"
|
git commit -e -m "sample"
|
||||||
head -n 7 .git/COMMIT_EDITMSG >actual &&
|
head -n 7 .git/COMMIT_EDITMSG >actual
|
||||||
|
'
|
||||||
|
|
||||||
|
test_expect_success C_LOCALE_OUTPUT 'author different from committer: output' '
|
||||||
test_cmp expect actual
|
test_cmp expect actual
|
||||||
'
|
'
|
||||||
|
|
||||||
@ -264,7 +269,10 @@ test_expect_success C_LOCALE_OUTPUT 'committer is automatic' '
|
|||||||
test_must_fail git commit -e -m "sample"
|
test_must_fail git commit -e -m "sample"
|
||||||
) &&
|
) &&
|
||||||
head -n 8 .git/COMMIT_EDITMSG | \
|
head -n 8 .git/COMMIT_EDITMSG | \
|
||||||
sed "s/^# Committer: .*/# Committer:/" >actual &&
|
sed "s/^# Committer: .*/# Committer:/" >actual
|
||||||
|
'
|
||||||
|
|
||||||
|
test_expect_success C_LOCALE_OUTPUT 'committer is automatic: output' '
|
||||||
test_cmp expect actual
|
test_cmp expect actual
|
||||||
'
|
'
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user