[PATCH v3] gitk: Add a "Save file as" menu item

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Previously, there was no easy way to save a particular file from the
currently selected revision.

This patch adds a menu item "Save file as" to the file list popup
menu, which opens a file selection dialog to determine the name under
which a file should be saved.  The default filename is of the form
"[shortid] basename".  If the current revision is the index, the
default pattern is of the form "[index] basename".  This works for
both, the "Patch" and "Tree" view.  The menu item is disabled for the
"local uncommitted changes" fake revision.

Signed-off-by: Andreas Amann <andreas.amann@xxxxxx>
---
 gitk | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

The previous version V2 of the patch did not work, when gitk was started
in a subdirectory of the repo and a file was saved from tree view.  This
is now fixed. 

diff --git a/gitk b/gitk
index 5cd00d8..6b66f18 100755
--- a/gitk
+++ b/gitk
@@ -2595,6 +2595,7 @@ proc makewindow {} {
 	{mc "Highlight this too" command {flist_hl 0}}
 	{mc "Highlight this only" command {flist_hl 1}}
 	{mc "External diff" command {external_diff}}
+	{mc "Save file as" command {save_file_as}}
 	{mc "Blame parent commit" command {external_blame 1}}
     }
     $flist_menu configure -tearoff 0
@@ -3378,6 +3379,7 @@ proc sel_flist {w x y} {
 proc pop_flist_menu {w X Y x y} {
     global ctext cflist cmitmode flist_menu flist_menu_file
     global treediffs diffids
+    global nullid
 
     stopfinding
     set l [lindex [split [$w index "@$x,$y"] "."] 0]
@@ -3395,6 +3397,12 @@ proc pop_flist_menu {w X Y x y} {
     }
     # Disable "External diff" item in tree mode
     $flist_menu entryconf 2 -state $xdiffstate
+    set savefilestate "normal"
+    if {[lindex $diffids 0] eq $nullid} {
+	set savefilestate "disabled"
+    }
+    # Disable "Save file as" item "local uncommited changes" revision
+    $flist_menu entryconf 3 -state $savefilestate
     tk_popup $flist_menu $X $Y
 }
 
@@ -3496,6 +3504,34 @@ proc external_diff_get_one_file {diffid filename diffdir} {
 	       "revision $diffid"]
 }
 
+proc save_file_as {} {
+    global nullid nullid2
+    global flist_menu_file cmitmode
+    global diffids
+
+    set diffid [lindex $diffids 0]
+    if {$diffid == $nullid} {
+	return
+    } elseif {$diffid == $nullid2} {
+	set diffidtext [mc "index"]
+	set diffid ""
+	set whattext $diffidtext
+    } else {
+	set diffidtext [shortids $diffid]
+	set whattext "[mc "revision"] $diffidtext"
+    }
+    set diffid $diffid:
+    if {$cmitmode eq "tree"} {
+	set diffid $diffid./
+    }
+    set difffile "\[$diffidtext\] [file tail $flist_menu_file]"
+    set difffile [tk_getSaveFile -initialfile $difffile -title [mc "Save file as"] -parent .]
+    if {$difffile eq {}} {
+	return
+    }
+    save_file_from_commit $diffid$flist_menu_file $difffile $whattext
+}
+
 proc external_diff {} {
     global nullid nullid2
     global flist_menu_file
-- 
1.8.3.3


Andreas Amann <a.amann@xxxxxx> writes:
> Previously, there was no easy way to save a particular file from the
> currently selected revision.
>
> This patch adds a menu item "Save file as" to the file list popup
> menu, which opens a file selection dialog to determine the name under
> which a file should be saved.  The default filename is of the form
> "[shortid] basename".  If the current revision is the index, the
> default pattern is of the form "[index] basename".  This works for
> both, the "Patch" and "Tree" view.  The menu item is disabled for the
> "local uncommitted changes" fake revision.
>
> Signed-off-by: Andreas Amann <andreas.amann@xxxxxx>
> ---
>  gitk | 32 ++++++++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
>
> diff --git a/gitk b/gitk
> index 5cd00d8..b5a70b5 100755
> --- a/gitk
> +++ b/gitk
> @@ -2595,6 +2595,7 @@ proc makewindow {} {
>  	{mc "Highlight this too" command {flist_hl 0}}
>  	{mc "Highlight this only" command {flist_hl 1}}
>  	{mc "External diff" command {external_diff}}
> +	{mc "Save file as" command {save_file_as}}
>  	{mc "Blame parent commit" command {external_blame 1}}
>      }
>      $flist_menu configure -tearoff 0
> @@ -3378,6 +3379,7 @@ proc sel_flist {w x y} {
>  proc pop_flist_menu {w X Y x y} {
>      global ctext cflist cmitmode flist_menu flist_menu_file
>      global treediffs diffids
> +    global nullid
>  
>      stopfinding
>      set l [lindex [split [$w index "@$x,$y"] "."] 0]
> @@ -3395,6 +3397,12 @@ proc pop_flist_menu {w X Y x y} {
>      }
>      # Disable "External diff" item in tree mode
>      $flist_menu entryconf 2 -state $xdiffstate
> +    set savefilestate "normal"
> +    if {[lindex $diffids 0] eq $nullid} {
> +	set savefilestate "disabled"
> +    }
> +    # Disable "Save file as" item "local uncommited changes" revision
> +    $flist_menu entryconf 3 -state $savefilestate
>      tk_popup $flist_menu $X $Y
>  }
>  
> @@ -3496,6 +3504,30 @@ proc external_diff_get_one_file {diffid filename diffdir} {
>  	       "revision $diffid"]
>  }
>  
> +proc save_file_as {} {
> +    global nullid nullid2
> +    global flist_menu_file
> +    global diffids
> +
> +    set diffid [lindex $diffids 0]
> +    if {$diffid == $nullid} {
> +	return
> +    } elseif {$diffid == $nullid2} {
> +	set diffidtext [mc "index"]
> +	set diffid ""
> +	set whattext $diffidtext
> +    } else {
> +	set diffidtext [shortids $diffid]
> +	set whattext "[mc "revision"] $diffidtext"
> +    }
> +    set difffile "\[$diffidtext\] [file tail $flist_menu_file]"
> +    set difffile [tk_getSaveFile -initialfile $difffile -title [mc "Save file as"] -parent .]
> +    if {$difffile eq {}} {
> +	return
> +    }
> +    save_file_from_commit $diffid:$flist_menu_file $difffile $whattext
> +}
> +
>  proc external_diff {} {
>      global nullid nullid2
>      global flist_menu_file
> -- 
> 1.8.3.3

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]