From b2a52531a69af34d25cd9e5ee0e7b42396093375 Mon Sep 17 00:00:00 2001 From: Nico Weber Date: Wed, 21 Aug 2019 16:03:34 +0000 Subject: [PATCH] git-llvm: Make push --force suppress error on nothing to commit as well git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@369544 91177308-0d34-0410-b5e6-96231b3b80d8 --- utils/git-svn/git-llvm | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/utils/git-svn/git-llvm b/utils/git-svn/git-llvm index e30d17c78a1..549a45558dd 100755 --- a/utils/git-svn/git-llvm +++ b/utils/git-svn/git-llvm @@ -209,11 +209,8 @@ def get_revs_to_push(rev_range): # Use git show rather than some plumbing command to figure out which revs # are in rev_range because it handles single revs (HEAD^) and ranges # (foo..bar) like we want. - revs = git('show', '--reverse', '--quiet', + return git('show', '--reverse', '--quiet', '--pretty=%h', rev_range).splitlines() - if not revs: - die('Nothing to push: No revs in range %s.' % rev_range) - return revs def clean_svn(svn_repo): @@ -462,6 +459,10 @@ def cmd_push(args): rev_range = args.rev_range dry_run = args.dry_run revs = get_revs_to_push(rev_range) + + if not args.force and not revs: + die('Nothing to push: No revs in range %s.' % rev_range) + log('%sPushing %d %s commit%s:\n%s' % ('[DryRun] ' if dry_run else '', len(revs), 'split-repo (%s)' % split_repo_path @@ -471,7 +472,7 @@ def cmd_push(args): for c in revs))) # Ask confirmation if multiple commits are about to be pushed - if not args.force and len(revs) != 1: + if not args.force and len(revs) > 1: if not ask_confirm("Are you sure you want to create %d commits?" % len(revs)): die("Aborting") -- 2.11.4.GIT