Re: [PATCH 1/2] t/t1417: test symbolic-ref effects on ref logs

2 messages, 2 authors, 2021-02-01 · open the first message on its own page

Re: [PATCH 1/2] t/t1417: test symbolic-ref effects on ref logs

From: Junio C Hamano <hidden>
Date: 2021-01-31 00:06:24

"Kyle J. McKay" [off-list ref] writes:
On Jan 30, 2021, at 11:56, Junio C Hamano wrote:
quoted
The said commit came as part of this topic, ...

https://lore.kernel.org/git/pull.669.v2.git.1594401593.gitgitgadget@gmail.com/

... so I've added the true author of it on the Cc: list.
Out of curiosity, if Han-Wen Nienhuys is the true author of commit
523fa69c36744ae6 why is it that you are both the committer and author  
of that commit in the commit's header?
See how the e-mail message was formatted in that thread.  I just ran
"am" on it (which makes me responsible for committing), and the
authorship comes from the "From:" that was in the body.  I suspect
he may have based the patch on some of the "how about doing it like
so" suggestions I made during an earlier discussion and wanted to
give me credit for the input, but I do not remember the context the
patch was originally written in X-<.


Re: [PATCH 1/2] t/t1417: test symbolic-ref effects on ref logs

From: Han-Wen Nienhuys <hidden>
Date: 2021-02-01 11:10:17

On Sun, Jan 31, 2021 at 12:48 AM Junio C Hamano [off-list ref] wrote:
quoted
On Jan 30, 2021, at 11:56, Junio C Hamano wrote:
quoted
The said commit came as part of this topic, ...

https://lore.kernel.org/git/pull.669.v2.git.1594401593.gitgitgadget@gmail.com/

... so I've added the true author of it on the Cc: list.
Out of curiosity, if Han-Wen Nienhuys is the true author of commit
523fa69c36744ae6 why is it that you are both the committer and author
of that commit in the commit's header?
See how the e-mail message was formatted in that thread.  I just ran
"am" on it (which makes me responsible for committing), and the
authorship comes from the "From:" that was in the body.  I suspect
he may have based the patch on some of the "how about doing it like
so" suggestions I made during an earlier discussion and wanted to
give me credit for the input, but I do not remember the context the
patch was originally written in X-<.
The classic reflog format doesn't allow '\n' in messages, but
different parts of the code did try to write '\n'. This patch was
supposed to sanitize the messages in a central location, so alternate
ref backends do not trigger spurious differences in how reflogs are
represented.

Your patch says
has changed in an unexpected way.
Can you make the expectations and current behavior explicit?

-- 
Han-Wen Nienhuys - Google Munich
I work 80%. Don't expect answers from me on Fridays.
--
Google Germany GmbH, Erika-Mann-Strasse 33, 80636 Munich
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschäftsführer: Paul Manicle, Halimah DeLaine Prado
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help