From: Phillip Wood Date: Fri, 23 Oct 2020 17:32:51 GMT Subject: Re: committer-date-is-author-date flag removes email in "Commit" Message-ID: <58e83ad2-14a2-5ace-c1fd-8224625a0f97@gmail.com> In-Reply-To: On 23/10/2020 16:23, Junio C Hamano wrote: > Jeff King writes: > >> On Fri, Oct 23, 2020 at 11:18:51AM +0530, VenomVendor wrote: >> >>> What did you do before the bug happened? (Steps to reproduce your issue) >>> * Create empty repo using `git init` >>> * Make few commits, at least two >>> * execute `git log --format=fuller` >>> * Notice the log, with "Author", "AuthorDate", "Commit", "CommitDate" >>> * Note, "Commit" >>> * execute `git rebase --committer-date-is-author-date HEAD~1` >>> * execute `git log --format=fuller` >>> * Note, email from "Commit" is empty <> >> >> Thanks for a clear report. I was able to easily reproduce the problem. >> There are actually two related bugs here, and they're both regressions >> in v2.29.0. >> >> [1/3]: t3436: check --committer-date-is-author-date result more carefully >> [2/3]: am: fix broken email with --committer-date-is-author-date >> [3/3]: rebase: fix broken email with --committer-date-is-author-date > > Thanks for taking quick care of this. It counts as an embarrasing > brown-paper-bag bug; it is a bit surprising that nobody noticed it > while the original change was discussed. It is indeed embarrassing. That change only appeared in the last round of patches and unfortunately I think most people had stopped looking them it by then as their comments had been addressed in previous rounds. > I wonder if we even needed to do the original change to begin with > (stopping to export means not giving information to the hooks), but > that is a separate matter. I think the main motivation was to stop polluting the environment of exec commands Best Wishes Phillip > Will take a look and queue. Thanks. > >> >> builtin/am.c | 4 ++-- >> sequencer.c | 2 +- >> t/t3436-rebase-more-options.sh | 4 ++-- >> 3 files changed, 5 insertions(+), 5 deletions(-) >> >> -Peff