Commit 074c2afe authored by tandrii@chromium.org's avatar tandrii@chromium.org

Revert of Gerrit git cl: temporary disable cc option on upload. (patchset #1...

Revert of Gerrit git cl: temporary disable cc option on upload. (patchset #1 id:1 of https://codereview.chromium.org/1895863002/ )

Reason for revert:
We've whitelisted cc-ed emails in Gerrit, and so let's try setting cc= again.

Original issue's description:
> Gerrit git cl: temporary disable cc option on upload.
> 
> Gerrit doesn't allow arbitrary non-member emails for now,
> thus removign this feature until either Gerrit allows it,
> or we have a per-repo opt-in way.
> 
> TBR=andybons@chromium.org
> BUG=604377
> 
> Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299988

TBR=andybons@chromium.org,sergiyb@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=604377

Review-Url: https://codereview.chromium.org/2034143002

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@300694 0039d316-1c4b-4281-b951-d872f2087c98
parent d389f5c5
......@@ -2498,10 +2498,7 @@ class _GerritChangelistImpl(_ChangelistCodereviewBase):
cc.extend(options.cc)
cc = filter(None, cc)
if cc:
# refspec_opts.extend('cc=' + email.strip() for email in cc)
# TODO(tandrii): enable this back. http://crbug.com/604377
print('WARNING: Gerrit doesn\'t yet support cc-ing arbitrary emails.\n'
' Ignoring cc-ed emails. See http://crbug.com/604377.')
refspec_opts.extend('cc=' + email.strip() for email in cc)
if change_desc.get_reviewers():
refspec_opts.extend('r=' + email.strip()
......
......@@ -821,18 +821,16 @@ class TestGitCl(TestCase):
expected_upstream_ref + '..' + ref_to_push],), ''),
((['git', 'config', 'rietveld.cc'],), '')
]
# Add cc from watch list.
# TODO(tandrii): bring this back after http://crbug.com/604377.
# if ref_suffix == '':
# ref_suffix = '%cc=joe@example.com'
# else:
# ref_suffix += ',cc=joe@example.com'
notify_suffix = 'notify=%s' % ('ALL' if notify else 'NONE')
if ref_suffix:
ref_suffix += ',' + notify_suffix
else:
ref_suffix = '%' + notify_suffix
# Add cc from watch list.
ref_suffix += ',cc=joe@example.com'
if reviewers:
ref_suffix += ',' + ','.join('r=%s' % email
for email in sorted(reviewers))
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment