git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 5/5] rebase: fix cherry-pick invocations

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
May 28, 2013, 13:29 UTC
Message-ID
<1369747757-10192-6-git-send-email-felipe.contreras@gmail.com>
In-Reply-To
<1369747757-10192-1-git-send-email-felipe.contreras@gmail.com>
So that all the tests pass.
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 git-rebase--cherry.sh | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/git-rebase--cherry.sh b/git-rebase--cherry.sh
index ca78b1b..c3a2ac9 100644
--- a/git-rebase--cherry.sh
+++ b/git-rebase--cherry.sh
@@ -23,11 +23,26 @@ test -n "$rebase_root" && root_flag=--root
 mkdir "$state_dir" || die "Could not create temporary $state_dir"
 : > "$state_dir"/cherry || die "Could not mark as cherry"
 
+if test -n "$rebase_root"
+then
+	revisions="$onto...$orig_head"
+else
+	revisions="$upstream...$orig_head"
+fi
+
 # we have to do this the hard way.  git format-patch completely squashes
 # empty commits and even if it didn't the format doesn't really lend
 # itself well to recording empty patches.  fortunately, cherry-pick
 # makes this easy
-git cherry-pick --allow-empty "$revisions"
+if test -n "$keep_empty"
+then
+	extra="--allow-empty"
+else
+	extra="--skip-empty --cherry-pick"
+fi
+test -n "$GIT_QUIET" && extra="$extra -q"
+test -z "$force_rebase" && extra="$extra --ff"
+git cherry-pick --no-merges --right-only --topo-order --do-walk --copy-notes $extra "$revisions"
 ret=$?
 
 if test 0 != $ret
-- 
1.8.3.rc3.312.g47657de
Previous: Felipe ContrerasNext: Junio C Hamano
Message 11 of 18 in “rebase: improve the keep-empty”
  1. 0/5 rebase: improve the keep-emptyFelipe Contreras, May 28, 2013
  2. 1/5 rebase: split the cherry-pick stuffFelipe Contreras, May 28, 2013
  3. Junio C HamanoMay 28, 2013
  4. 2/5 rebase: fix 'cherry' mode storageFelipe Contreras, May 28, 2013
  5. Junio C HamanoMay 28, 2013
  6. Felipe ContrerasMay 29, 2013
  7. 3/5 rebase: fix sequence continuationFelipe Contreras, May 28, 2013
  8. Junio C HamanoMay 28, 2013
  9. Felipe ContrerasMay 29, 2013
  10. 4/5 rebase: fix abort of cherry modeFelipe Contreras, May 28, 2013
  11. 5/5 rebase: fix cherry-pick invocationsFelipe Contreras, May 28, 2013
  12. Junio C HamanoMay 28, 2013
  13. Junio C HamanoMay 28, 2013
  14. Felipe ContrerasMay 29, 2013
  15. Felipe ContrerasMay 29, 2013
  16. Felipe ContrerasMay 28, 2013
  17. Martin von ZweigbergkMay 28, 2013
  18. Felipe ContrerasMay 29, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.