Skip to content

Commit 963d02a

Browse files
peffgitster
authored andcommitted
t7415: remove out-dated comment about translation
Since GETTEXT_POISON does not exist anymore, there is no point warning people about whether we should use test_i18ngrep. This is doubly confusing because the comment was describing why it was OK to use grep, but it got caught up in the mass conversion of 674ba34 (fsck: mark strings for translation, 2018-11-10). Note there are other uses of test_i18ngrep in this script which are now obsolete; I'll save those for a mass-cleanup. My goal here was just to fix the confusing comment in code I'm about to refactor. Signed-off-by: Jeff King <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 7e39198 commit 963d02a

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

t/t7415-submodule-names.sh

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -151,10 +151,9 @@ test_expect_success 'fsck detects symlinked .gitmodules file' '
151151
} | git mktree &&
152152
153153
# Check not only that we fail, but that it is due to the
154-
# symlink detector; this grep string comes from the config
155-
# variable name and will not be translated.
154+
# symlink detector
156155
test_must_fail git fsck 2>output &&
157-
test_i18ngrep gitmodulesSymlink output
156+
grep gitmodulesSymlink output
158157
)
159158
'
160159

0 commit comments

Comments
 (0)