4.5 months and counting... :( - Orgad On Sun, Jul 10, 2016 at 7:36 AM, Orgad Shaneh <orgads@xxxxxxxxx> wrote: > It's been over 2 months. Can anyone please review and merge it? > > Thanks. > - Orgad > > On Wed, May 18, 2016 at 9:12 AM, Orgad Shaneh <orgads@xxxxxxxxx> wrote: >> ping? >> >> On Thu, May 5, 2016 at 8:22 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: >>> Pat, we haven't heard from you for a long time. Are you still >>> around and interested in helping us by maintaining git-gui? >>> >>> Otherwise we may have to start recruiting a volunteer or two to take >>> this over. >>> >>> Thanks. >>> >>> Orgad Shaneh <orgads@xxxxxxxxx> writes: >>> >>>> git commit --amend preserves the author details unless --reset-author is >>>> given. >>>> >>>> git-gui discards the author details on amend. >>>> >>>> Fix by reading the author details along with the commit message, and >>>> setting the appropriate environment variables required for preserving >>>> them. >>>> >>>> Reported long ago in the mailing list[1]. >>>> >>>> [1] http://article.gmane.org/gmane.comp.version-control.git/243921 >>>> >>>> Signed-off-by: Orgad Shaneh <orgads@xxxxxxxxx> >>>> --- >>>> git-gui/lib/commit.tcl | 19 +++++++++++++++++++ >>>> 1 file changed, 19 insertions(+) >>>> >>>> diff --git a/git-gui/lib/commit.tcl b/git-gui/lib/commit.tcl >>>> index 864b687..60edf99 100644 >>>> --- a/git-gui/lib/commit.tcl >>>> +++ b/git-gui/lib/commit.tcl >>>> @@ -1,8 +1,13 @@ >>>> # git-gui misc. commit reading/writing support >>>> # Copyright (C) 2006, 2007 Shawn Pearce >>>> >>>> +set author_name "" >>>> +set author_email "" >>>> +set author_date "" >>>> + >>>> proc load_last_commit {} { >>>> global HEAD PARENT MERGE_HEAD commit_type ui_comm >>>> + global author_name author_email author_date >>>> global repo_config >>>> >>>> if {[llength $PARENT] == 0} { >>>> @@ -34,6 +39,10 @@ You are currently in the middle of a merge that has not been fully completed. Y >>>> lappend parents [string range $line 7 end] >>>> } elseif {[string match {encoding *} $line]} { >>>> set enc [string tolower [string range $line 9 end]] >>>> + } elseif {[regexp "author (.*)\\s<(.*)>\\s(\\d.*$)" $line all name email time]} { >>>> + set author_name $name >>>> + set author_email $email >>>> + set author_date $time >>>> } >>>> } >>>> set msg [read $fd] >>>> @@ -107,8 +116,12 @@ proc do_signoff {} { >>>> >>>> proc create_new_commit {} { >>>> global commit_type ui_comm >>>> + global author_name author_email author_date >>>> >>>> set commit_type normal >>>> + set author_name "" >>>> + set author_email "" >>>> + set author_date "" >>>> $ui_comm delete 0.0 end >>>> $ui_comm edit reset >>>> $ui_comm edit modified false >>>> @@ -327,6 +340,7 @@ proc commit_committree {fd_wt curHEAD msg_p} { >>>> global ui_comm selected_commit_type >>>> global file_states selected_paths rescan_active >>>> global repo_config >>>> + global env author_name author_email author_date >>>> >>>> gets $fd_wt tree_id >>>> if {[catch {close $fd_wt} err]} { >>>> @@ -366,6 +380,11 @@ A rescan will be automatically started now. >>>> } >>>> } >>>> >>>> + if {$author_name ne ""} { >>>> + set env(GIT_AUTHOR_NAME) $author_name >>>> + set env(GIT_AUTHOR_EMAIL) $author_email >>>> + set env(GIT_AUTHOR_DATE) $author_date >>>> + } >>>> # -- Create the commit. >>>> # >>>> set cmd [list commit-tree $tree_id]