Looks good. Committed to rbtools master as 9137e9e
Do not run git remote show just because of remote URL
Review Request #1377 — Created Feb. 1, 2010 and submitted
Current post-review's GitClient calls "git remote show" to determine URL of upstream repository and parses its output. This patch replaces this with fetching the URL from local configuration, which spares a few moments of time since "git remote show" has to communicate with the remote in order to determine list of remote branches (which is subsequently discarded by the script).
Tested locally, seems to work fine. (I used it to submit this review request).