AW: [PATCH v2] gitk: added external diff file rename detection

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

 



> -----Ursprüngliche Nachricht-----
> Von: ToBoMi via GitGitGadget <gitgitgadget@xxxxxxxxx>
> Gesendet: Freitag, 6. September 2024 09:28
> An: git@xxxxxxxxxxxxxxx
> Cc: Boesch, Tobias <tobias.boesch@xxxxxxxxx>; Boesch, Tobias
> <tobias.boesch@xxxxxxxxx>
> Betreff: [PATCH v2] gitk: added external diff file rename detection
>
> From: Tobias Boesch <tobias.boesch@xxxxxxxxx>
>
> * If a file was renamed between commits and an external diff is started
>   through gitk on the THE ORIGINAL FILE NAME (not the renamed one),
>   gitk was unable to open the renamed file in the external diff editor.
>   It failed to fetch the renamed file from git, because it fetched it
>   with the original path in contrast to using the renamed path
> * gitk now detects the rename and opens the external diff with the
>   original and the RENAMED file instead of no file (it is able to
>   fetch the renamed file now from git with the renamed path/filename)
> * Since git doesn't destinguish between move or rename this also works
>   for moved files
> * External diff detection and usage is optional and has to be enabled in
>   gitk settings
> * External rename detection ist marked EXPERIMENTAL in the settings
>   and disabled by default

It would be nice to have this without an additional setting.
Just enable the feature and remove the option.

> * Showing the renamed file doesn't work when THE RENAMED FILE is selected
>   in gitk and an external diff ist started on that file, because the
>   selected file is not renamed in that commit. It already IS the renamed
>   file.

gitk already shows the exact same diff for both file entries, the file that is renamed and the renamed file.
This is why it would be okay to show the same external diff for both file entries.
It should be possible from the current code change to also gather the file names the other way round, that is to gather the file path of the file to be renamed from the renamed file path.
Since gitk doesn't change the diff depending on the file that is selected, the external diff should not switch sides depending on the file being selected.

>
> Signed-off-by: Tobias Boeesch <tobias.boesch@xxxxxxxxx>
> ---
>     gitk: added external diff file rename detection
>
>     Changes since v1:
>
>      * Commit message ident
>      * Commit message line length
>
> Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-
> 1774%2FToBoMi%2Fdetect_renamed_files_when_opening_diff-v2
> Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-
> 1774/ToBoMi/detect_renamed_files_when_opening_diff-v2
> Pull-Request: https://github.com/gitgitgadget/git/pull/1774
>
> Range-diff vs v1:
>
>  1:  4ff4aec82fe ! 1:  6209080cad6 gitk: added external diff file rename
> detection
>      @@
>        ## Metadata ##
>      -Author: deboeto <tobias.boesch@xxxxxxxxx>
>      +Author: Tobias Boesch <tobias.boesch@xxxxxxxxx>
>
>        ## Commit message ##
>           gitk: added external diff file rename detection
>
>      -    * If a file was renamed between commits and
>      -        an external diff is started through gitk
>      -        on the THE ORIGINAL FILE NAME (not the
>      -        renamed one), gitk was unable to open
>      -        the renamed file in the external diff
>      -        editor.
>      -        It failed to fetch the renamed file from
>      -        git, because it fetched it with the original
>      -        path in contrast to using the renamed path
>      -    *   gitk now detects the rename and opens the
>      -        external diff with the original and the RENAMED
>      -        file instead of no file (it is able to fetch
>      -        the renamed file now from git with the renamed
>      -        path/filename)
>      -    * Since git doesn't destinguish between move or
>      -        rename this also works for moved files
>      -    * External diff detection and usage is optional
>      -        and has to be enabled in gitk settings
>      -    * External rename detection ist marked
>      -        EXPERIMENTAL in the settings and disabled
>      -        by default
>      -    * Showing the renamed file doesn't work when THE
>      -        RENAMED FILE is selected in gitk and an
>      -        external diff ist started on that file,
>      -        because the selected file is not renamed in
>      -        that commit. It already IS the renamed file.
>      +    * If a file was renamed between commits and an external diff is started
>      +      through gitk on the THE ORIGINAL FILE NAME (not the renamed one),
>      +      gitk was unable to open the renamed file in the external diff editor.
>      +      It failed to fetch the renamed file from git, because it fetched it
>      +      with the original path in contrast to using the renamed path
>      +    * gitk now detects the rename and opens the external diff with the
>      +      original and the RENAMED file instead of no file (it is able to
>      +      fetch the renamed file now from git with the renamed path/filename)
>      +    * Since git doesn't destinguish between move or rename this also works
>      +      for moved files
>      +    * External diff detection and usage is optional and has to be enabled in
>      +      gitk settings
>      +    * External rename detection ist marked EXPERIMENTAL in the settings
>      +      and disabled by default
>      +    * Showing the renamed file doesn't work when THE RENAMED FILE is
> selected
>      +      in gitk and an external diff ist started on that file, because the
>      +      selected file is not renamed in that commit. It already IS the renamed
>      +      file.
>
>      -    Signed-off-by: deboeto <tobias.boesch@xxxxxxxxx>
>      +    Signed-off-by: Tobias Boeesch <tobias.boesch@xxxxxxxxx>
>
>        ## gitk-git/gitk ##
>       @@ gitk-git/gitk: proc external_diff_get_one_file {diffid filename diffdir} {
>
>
>  gitk-git/gitk | 54 ++++++++++++++++++++++++++++++++++++++++++++--
> -----
>  1 file changed, 47 insertions(+), 7 deletions(-)
>
> diff --git a/gitk-git/gitk b/gitk-git/gitk index 7a087f123d7..f7427f6d3f2
> 100755
> --- a/gitk-git/gitk
> +++ b/gitk-git/gitk
> @@ -3662,11 +3662,33 @@ proc external_diff_get_one_file {diffid filename
> diffdir} {
>                 "revision $diffid"]
>  }
>
> +proc check_for_renames_in_diff {filepath} {
> +    global ctext
> +
> +    set renamed_filenames [list {}]
> +    set filename [file tail $filepath]
> +    set rename_from_text_length 12
> +    set rename_to_text_length 10
> +    set reg_expr_rename_from {^rename from (.*$filename)}
> +    set reg_expr_rename_from [subst -nobackslashes -nocommands
> $reg_expr_rename_from]
> +    set reg_expr_rename_to {^rename to (.*)}
> +    set rename_from_text_index [$ctext search -elide -regexp --
> $reg_expr_rename_from 0.0]
> +    if { ($rename_from_text_index != {})} {
> +        set rename_to_text_index [$ctext search -elide -regexp --
> $reg_expr_rename_to $rename_from_text_index]
> +        if { ($rename_from_text_index != {}) && ($rename_to_text_index != {}) }
> {
> +            lappend renamed_filenames [$ctext get "$rename_from_text_index +
> $rename_from_text_length chars" "$rename_from_text_index lineend"]
> +            lappend renamed_filenames [$ctext get "$rename_to_text_index +
> $rename_to_text_length chars" "$rename_to_text_index lineend"]
> +        }
> +    }
> +    return $renamed_filenames
> +}
> +
>  proc external_diff {} {
>      global nullid nullid2
>      global flist_menu_file
>      global diffids
>      global extdifftool
> +    global file_rename_detection
>
>      if {[llength $diffids] == 1} {
>          # no reference commit given
> @@ -3692,8 +3714,21 @@ proc external_diff {} {
>      if {$diffdir eq {}} return
>
>      # gather files to diff
> -    set difffromfile [external_diff_get_one_file $diffidfrom $flist_menu_file
> $diffdir]
> -    set difftofile [external_diff_get_one_file $diffidto $flist_menu_file $diffdir]
> +    if {$file_rename_detection} {
> +        set renamed_filenames [check_for_renames_in_diff $flist_menu_file]
> +        set rename_from_filename [lindex $renamed_filenames 1]
> +        set rename_to_filename [lindex $renamed_filenames 2]
> +        if { ($rename_from_filename != {}) && ($rename_to_filename != {}) } {
> +            set difffromfile [external_diff_get_one_file $diffidfrom
> $rename_from_filename $diffdir]
> +            set difftofile [external_diff_get_one_file $diffidto $rename_to_filename
> $diffdir]
> +        } else {
> +            set difffromfile [external_diff_get_one_file $diffidfrom $flist_menu_file
> $diffdir]
> +            set difftofile [external_diff_get_one_file $diffidto $flist_menu_file
> $diffdir]
> +        }
> +    } else {
> +        set difffromfile [external_diff_get_one_file $diffidfrom $flist_menu_file
> $diffdir]
> +        set difftofile [external_diff_get_one_file $diffidto $flist_menu_file
> $diffdir]
> +    }
>
>      if {$difffromfile ne {} && $difftofile ne {}} {
>          set cmd [list [shellsplit $extdifftool] $difffromfile $difftofile] @@ -
> 11577,7 +11612,7 @@ proc create_prefs_page {w} {  proc prefspage_general
> {notebook} {
>      global NS maxwidth maxgraphpct showneartags showlocalchanges
>      global tabstop limitdiffs autoselect autosellen extdifftool perfile_attrs
> -    global hideremotes want_ttk have_ttk maxrefs web_browser
> +    global hideremotes want_ttk have_ttk maxrefs web_browser
> + file_rename_detection
>
>      set page [create_prefs_page $notebook.general]
>
> @@ -11639,12 +11674,16 @@ proc prefspage_general {notebook} {
>      grid $page.lgen - -sticky w -pady 10
>      ${NS}::checkbutton $page.want_ttk -variable want_ttk \
>          -text [mc "Use themed widgets"]
> +    ${NS}::checkbutton $page.file_rename_detection -variable
> file_rename_detection \
> +        -text [mc "Use ext diff file rename detection"]
> +    ${NS}::label $page.file_rename_detection_note -text [mc
> + "(EXPERIMENTAL\nTries to find the file path of a\nrenamed file in
> + external diff)"]
>      if {$have_ttk} {
>          ${NS}::label $page.ttk_note -text [mc "(change requires restart)"]
>      } else {
>          ${NS}::label $page.ttk_note -text [mc "(currently unavailable)"]
>      }
>      grid x $page.want_ttk $page.ttk_note -sticky w
> +    grid x $page.file_rename_detection $page.file_rename_detection_note
> + -sticky w
>      return $page
>  }
>
> @@ -11725,7 +11764,7 @@ proc doprefs {} {
>      global oldprefs prefstop showneartags showlocalchanges
>      global uicolor bgcolor fgcolor ctext diffcolors selectbgcolor markbgcolor
>      global tabstop limitdiffs autoselect autosellen extdifftool perfile_attrs
> -    global hideremotes want_ttk have_ttk
> +    global hideremotes want_ttk have_ttk file_rename_detection
>
>      set top .gitkprefs
>      set prefstop $top
> @@ -11734,7 +11773,7 @@ proc doprefs {} {
>          return
>      }
>      foreach v {maxwidth maxgraphpct showneartags showlocalchanges \
> -                   limitdiffs tabstop perfile_attrs hideremotes want_ttk} {
> +                   limitdiffs tabstop perfile_attrs hideremotes
> + want_ttk file_rename_detection} {
>          set oldprefs($v) [set $v]
>      }
>      ttk_toplevel $top
> @@ -11860,7 +11899,7 @@ proc prefscan {} {
>      global oldprefs prefstop
>
>      foreach v {maxwidth maxgraphpct showneartags showlocalchanges \
> -                   limitdiffs tabstop perfile_attrs hideremotes want_ttk} {
> +                   limitdiffs tabstop perfile_attrs hideremotes
> + want_ttk file_rename_detection} {
>          global $v
>          set $v $oldprefs($v)
>      }
> @@ -12404,6 +12443,7 @@ set autoselect 1  set autosellen 40  set
> perfile_attrs 0  set want_ttk 1
> +set file_rename_detection 0
>
>  if {[tk windowingsystem] eq "aqua"} {
>      set extdifftool "opendiff"
> @@ -12498,7 +12538,7 @@ config_check_tmp_exists 50  set
> config_variables {
>      mainfont textfont uifont tabstop findmergefiles maxgraphpct maxwidth
>      cmitmode wrapcomment autoselect autosellen showneartags maxrefs
> visiblerefs
> -    hideremotes showlocalchanges datetimeformat limitdiffs uicolor want_ttk
> +    hideremotes showlocalchanges datetimeformat limitdiffs uicolor
> + want_ttk file_rename_detection
>      bgcolor fgcolor uifgcolor uifgdisabledcolor colors diffcolors mergecolors
>      markbgcolor diffcontext selectbgcolor foundbgcolor
> currentsearchhitbgcolor
>      extdifftool perfile_attrs headbgcolor headfgcolor headoutlinecolor
>
> base-commit: 2e7b89e038c0c888acf61f1b4ee5a43d4dd5e94c
> --
> gitgitgadget


-------------------------------------------------------------------------------------------------
imperial-Werke oHG, Sitz Bünde, Registergericht Bad Oeynhausen - HRA 4825




[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]

  Powered by Linux