Looks good. Thanks for the change!
Add --ignore-submodules to postreview.py
Review Request #3052 — Created April 9, 2012 and submitted
RB doesn't handle submodules; diffs that include submodule changes break diff uploading. Thus, postreview.py shouldn't include them. https://github.com/reviewboard/rbtools/pull/9
We use this patch in production at Yelp.