Chromium Code Reviews| Index: git_cl.py |
| diff --git a/git_cl.py b/git_cl.py |
| index 3c7cd3f315fac72e94414960910bbf7e16104d24..b05c8066af0ea507854f8a751878a8f42ce7f6df 100755 |
| --- a/git_cl.py |
| +++ b/git_cl.py |
| @@ -1737,12 +1737,15 @@ class _RietveldChangelistImpl(_ChangelistCodereviewBase): |
| return self.RpcServer().close_issue(self.GetIssue()) |
| def SetFlag(self, flag, value): |
| + return self.SetFlags({flag: value}) |
| + |
| + def SetFlags(self, flags): |
| """Patchset must match.""" |
|
Michael Achenbach
2016/07/06 07:47:59
fly-by nit: This is not a method comment, rather a
tandrii(chromium)
2016/07/06 10:44:03
Done.
|
| if not self.GetPatchset(): |
| DieWithError('The patchset needs to match. Send another patchset.') |
| try: |
| - return self.RpcServer().set_flag( |
| - self.GetIssue(), self.GetPatchset(), flag, value) |
| + return self.RpcServer().set_flags( |
| + self.GetIssue(), self.GetPatchset(), flags) |
| except urllib2.HTTPError as e: |
| if e.code == 404: |
| DieWithError('The issue %s doesn\'t exist.' % self.GetIssue()) |
| @@ -1793,7 +1796,8 @@ class _RietveldChangelistImpl(_ChangelistCodereviewBase): |
| elif new_state == _CQState.NONE: |
| self.SetFlag('commit', '0') |
| else: |
| - raise NotImplementedError() |
| + assert new_state == _CQState.DRY_RUN |
| + self.SetFlags({'commit': '1', 'cq_dry_run': '1'}) |
| def CMDPatchWithParsedIssue(self, parsed_issue_arg, reject, nocommit, |