diff --git a/git_cl.py b/git_cl.py index 26f04fd8c..5ff11fc15 100755 --- a/git_cl.py +++ b/git_cl.py @@ -2394,7 +2394,9 @@ def SendUpstream(parser, args, cmd): commit_desc = ChangeDescription(change_desc.description) if cl.GetIssue(): - commit_desc.append_footer('Review URL: %s' % cl.GetIssueURL()) + # Xcode won't linkify this URL unless there is a non-whitespace character + # after it. Add a period on a new line to circumvent this. + commit_desc.append_footer('Review URL: %s.' % cl.GetIssueURL()) if options.contributor: commit_desc.append_footer('Patch from %s.' % options.contributor) diff --git a/tests/git_cl_test.py b/tests/git_cl_test.py index 54878ebf3..cb87bdb4c 100755 --- a/tests/git_cl_test.py +++ b/tests/git_cl_test.py @@ -359,7 +359,7 @@ class TestGitCl(TestCase): ((['git', 'reset', '--soft', 'fake_ancestor_sha'],), ''), ((['git', 'commit', '-m', 'Issue: 12345\n\nR=john@chromium.org\n\n' - 'Review URL: https://codereview.example.com/12345'],), + 'Review URL: https://codereview.example.com/12345.'],), ''), ((['git', 'config', 'rietveld.force-https-commit-url'],), ''), ((['git',