From 4898ca5b67dae850d129e805bc18b52b69f3f4a0 Mon Sep 17 00:00:00 2001 From: Monty Taylor Date: Fri, 25 Jan 2013 11:57:06 +1100 Subject: [PATCH] Don't fetch remote outside of rebase flow. The assert_one_change code checks your changes against the set of changes in the branch that you're submitting to. It's not necessary to have the absolute latest changes to do the safety check ... the worst that will happen is that you'll be prompted about a patch that's already merged. However, if you run without -R, you'll get the update fetched anyway. If you run _WITH_ -R, then we're doing a network access that you don't need anyway. Change-Id: I466aa81a521f784d9afc1dbae914c68a33ff79f5 --- git-review | 1 - 1 file changed, 1 deletion(-) diff --git a/git-review b/git-review index 40ae4044..b919ad5f 100755 --- a/git-review +++ b/git-review @@ -453,7 +453,6 @@ def get_branch_name(target_branch): def assert_one_change(remote, branch, yes, have_hook): - update_remote(remote) branch_name = get_branch_name(branch) has_color = check_color_support() if has_color: