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

[PATCH (GITK) v3 4/6] gitk: Make cherry-pick call git-citool on conflicts.

From
Alexander Gavrilov <angavrilov@gmail.com>
Date
Nov 2, 2008, 18:59 UTC
Message-ID
<1225652389-22082-5-git-send-email-angavrilov@gmail.com>
In-Reply-To
<1225652389-22082-4-git-send-email-angavrilov@gmail.com>

Now that git-gui has facilities to help users resolve conflicts, it makes sense to launch it from other GUI tools when they happen.

Signed-off-by: Alexander Gavrilov <angavrilov@gmail.com>
---
 gitk |   40 +++++++++++++++++++++++++++++++++++++++-
 1 files changed, 39 insertions(+), 1 deletions(-)
diff --git a/gitk b/gitk
index f747c70..71f1b27 100755
--- a/gitk
+++ b/gitk
@@ -8110,6 +8110,32 @@ proc mkbrgo {top} {
     }
 }
 
+proc exec_citool {tool_args {baseid {}}} {
+    global commitinfo env
+
+    set save_env [array get env GIT_AUTHOR_*]
+
+    if {$baseid ne {}} {
+	if {![info exists commitinfo($baseid)]} {
+	    getcommit $baseid
+	}
+	set author [lindex $commitinfo($baseid) 1]
+	set date [lindex $commitinfo($baseid) 2]
+	if {[regexp {^\s*(\S.*\S|\S)\s*<(.*)>\s*$} \
+	            $author author name email]
+	    && $date ne {}} {
+	    set env(GIT_AUTHOR_NAME) $name
+	    set env(GIT_AUTHOR_EMAIL) $email
+	    set env(GIT_AUTHOR_DATE) $date
+	}
+    }
+
+    eval exec git citool $tool_args &
+
+    array unset env GIT_AUTHOR_*
+    array set env $save_env
+}
+
 proc cherrypick {} {
     global rowmenuid curview
     global mainhead mainheadid
@@ -8128,7 +8154,19 @@ proc cherrypick {} {
     # no error occurs, and exec takes that as an indication of error...
     if {[catch {exec sh -c "git cherry-pick -r $rowmenuid 2>&1"} err]} {
 	notbusy cherrypick
-	error_popup $err
+	if {[regexp -line \
+	    {Entry '(.*)' would be overwritten by merge} $err msg fname]} {
+	    error_popup [mc "Cherry-pick failed:
+file '%s' had local modifications.
+
+Please commit, reset or stash your changes." $fname]
+	} elseif {[regexp -line {^CONFLICT \(.*\):} $err msg]} {
+	    # Force citool to read MERGE_MSG
+	    file delete [file join [gitdir] "GITGUI_MSG"]
+	    exec_citool {} $rowmenuid
+	} else {
+	    error_popup $err
+	}
 	return
     }
     set newhead [exec git rev-parse HEAD]
-- 
1.6.0.3.15.gb8d36
Previous: Alexander GavrilovNext: Alexander Gavrilov
Message 5 of 21 in “Enhance popup dialogs in gitk.”
  1. 0/6 Enhance popup dialogs in gitk.Alexander Gavrilov, Nov 2, 2008
  2. 1/6 gitk: Add Return and Escape bindings to dialogs.Alexander Gavrilov, Nov 2, 2008
  3. 2/6 gitk: Make gitk dialog windows transient.Alexander Gavrilov, Nov 2, 2008
  4. 3/6 gitk: Add accelerators to frequently used menu commands.Alexander Gavrilov, Nov 2, 2008
  5. 4/6 gitk: Make cherry-pick call git-citool on conflicts.Alexander Gavrilov, Nov 2, 2008
  6. 5/6 gitk: Implement a user-friendly Edit View dialog.Alexander Gavrilov, Nov 2, 2008
  7. 6/6 gitk: Explicitly position popup windows.Alexander Gavrilov, Nov 2, 2008
  8. Paul MackerrasNov 7, 2008
  9. Alex RiesenNov 7, 2008
  10. Johannes SixtNov 7, 2008
  11. Alex RiesenNov 9, 2008
  12. Paul MackerrasNov 9, 2008
  13. Alexander GavrilovNov 9, 2008
  14. Paul MackerrasNov 10, 2008
  15. Alexander GavrilovNov 10, 2008
  16. Alexander GavrilovNov 11, 2008
  17. Paul MackerrasNov 7, 2008
  18. Paul MackerrasNov 7, 2008
  19. Alexander GavrilovNov 9, 2008
  20. Paul MackerrasNov 7, 2008
  21. Paul MackerrasNov 7, 2008

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.