If you add a footer like R=reviewer@chromium.org to your Gerrit CL, then `git cl upload` can crash after uploading your patch. The R= syntax causes the expected behavior of adding the reviewer(s) to the CL as expected, so I don't think this is my fault for confusing the tools.

Here's a CL where I use that syntax:

https://chromium-review.googlesource.com/c/546941/3

And here's the output from `git cl upload` that shows the stacktrace:

v8$ git cl upload
Running presubmit upload checks ...
Done processing /home/jwolfe/v8/v8/test/cctest/test-compiler.cc
Done processing /home/jwolfe/v8/v8/src/flag-definitions.h
Total errors found: 0
Total violating files: 0

Presubmit checks took 3.1s to calculate.

Presubmit checks passed.
 src/flag-definitions.h                         |  8 ++++----
 test/cctest/test-compiler.cc                   |  4 ++--
 test/message/paren_in_arg_string.out           |  8 ++++----
 test/mjsunit/harmony/async-generators-basic.js |  6 +++---
 test/mjsunit/regress/regress-2470.js           |  2 +-
 test/mjsunit/regress/regress-crbug-109362.js   | 11 ++++++-----
 test/mjsunit/regress/regress-crbug-663410.js   |  2 +-
 test/test262/test262.status                    |  3 ---
 8 files changed, 21 insertions(+), 23 deletions(-)
Title for patchset [Enable --harmony-function-tostring by default]: rebase
remote: Processing changes: updated: 1, done            
remote: 
remote: Updated Changes:        
remote:   https://chromium-review.googlesource.com/546941 Enable --harmony-function-tostring by default        
remote: 
To https://chromium.googlesource.com/v8/v8.git
 * [new branch]      dbc72d2de9e152e3f88597e9d214ee73392043e5 -> refs/for/refs/heads/master%notify=NONE,m=rebase
Traceback (most recent call last):
  File "/home/jwolfe/depot_tools/git_cl.py", line 6055, in <module>
    sys.exit(main(sys.argv[1:]))
  File "/home/jwolfe/depot_tools/git_cl.py", line 6037, in main
    return dispatcher.execute(OptionParser(), argv)
  File "/home/jwolfe/depot_tools/subcommand.py", line 252, in execute
    return command(parser, args[1:])
  File "/home/jwolfe/depot_tools/git_cl.py", line 4904, in CMDupload
    return cl.CMDUpload(options, args, orig_args)
  File "/home/jwolfe/depot_tools/git_cl.py", line 1673, in CMDUpload
    sys.stdout)
  File "/home/jwolfe/depot_tools/presubmit_support.py", line 1194, in DoPostUploadExecuter
    presubmit_script, filename, cl, change))
  File "/home/jwolfe/depot_tools/presubmit_support.py", line 1097, in ExecPresubmitScript
    return post_upload_hook(cl, change, OutputApi(False))
  File "<string>", line 371, in PostUploadHook
  File "/home/jwolfe/depot_tools/presubmit_support.py", line 309, in EnsureCQIncludeTrybotsAreAdded
    description, 'Cq-Include-Trybots')
  File "/home/jwolfe/depot_tools/git_footers.py", line 150, in remove_footer
    l for l in footer_lines if normalize_name(parse_footer(l)[0]) != key]
TypeError: 'NoneType' object has no attribute '__getitem__'