[DIM PATCH v2 2/2] dim: Fix create-branch dry run
Sean Paul
seanpaul at chromium.org
Mon May 14 17:10:33 UTC 2018
Using --dry-run for git push on a non-existant branch (since the command
above doesn't actually create the branch) fails with:
error: src refspec topic/dim_test does not match any.
error: failed to push some refs to <remote>
So instead of using --dry-run for git push, just echo the command.
Changes in v2:
- Added comment (Daniel)
Reviewed-by: Daniel Vetter <daniel.vetter at ffwll.ch>
Signed-off-by: Sean Paul <seanpaul at chromium.org>
---
dim | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/dim b/dim
index 7d8787d22c8b..71407eb034b4 100755
--- a/dim
+++ b/dim
@@ -1277,7 +1277,10 @@ function dim_create_branch
fi
$DRY git branch $branch $start
- git push $DRY_RUN $remote +$branch --set-upstream
+
+ # git push gives confusing error messages for non-existing
+ # branches, even with --dry-run, hence the even quieter $DRY
+ $DRY git push $remote +$branch --set-upstream
cd $DIM_PREFIX/drm-rerere
$DRY sed -i "s/^\() # DO NOT CHANGE THIS LINE\)$/\t\"$repo\t\t${branch//\//\\\/}\"\n\1/" $dim_integration_config
--
Sean Paul, Software Engineer, Google / Chromium OS
More information about the dim-tools
mailing list