Re: [PATCH v3 4/5] rebase -i: support --ignore-date
- From
Đoàn Trần Công Danh <congdanhqx@gmail.com>
- Date
- May 23, 2020, 15:52 UTC
- Message-ID
- <20200523155203.GA10163@danh.dev>
- In-Reply-To
- <bc6bc4d4-79cb-f788-deca-41a3735fcaae@gmail.com>
Hi Phillip
On 2020-05-23 16:43:39+0100, Phillip Wood <phillip.wood123@gmail.com> wrote:
Show 24 quoted lines
> > On 2020-05-21 11:14:54+0100, Phillip Wood <phillip.wood123@gmail.com> wrote: > > > +# Checking for +0000 in author time is enough since default > > > +# timezone is UTC, but the timezone used while committing > > > +# sets to +0530. > > > +test_expect_success '--ignore-date works with apply backend' ' > > > + git commit --amend --date="$GIT_AUTHOR_DATE" && > > > + git rebase --apply --ignore-date HEAD^ && > > > + git log -1 --pretty="format:%ai" >authortime && > > > > Those --pretty="format:%ai" won't print the newline character in my > > test environment. > > It looks like it won't print the newline if stdout isn't a tty. > > > > git log -1 --pretty=%ai > > > > doesn't have that issue. > > > > I think there're some grep out there considers file doesn't end with > > newline as non-text files. > > Yes it would be better to print the newline, thanks > > Junio - are you happy to fix this up (assuming there are no other issues) or > do you want a re-roll?
There're 11 invocation of git-log with "--pretty=format", in 2/5 and 4/5
I think it's worth to have a re-roll to avoid mistake.
-- Danh