Re: [GSOC PATCH 2/2] config: set comment_line_str to "#" when core.commentChar=auto
- From
Ayush Chandekar <ayu.chandekar@gmail.com>
- Date
- Jul 16, 2025, 11:04 UTC
- Message-ID
- <CAE7as+YxajFO0FfMe2wYpT9okYQoevZAghDD29d7E0P82-A_Hw@mail.gmail.com>
- In-Reply-To
- <xmqqcya1f2vr.fsf@gitster.g>
On Wed, Jul 16, 2025 at 5:00 AM Junio C Hamano <gitster@pobox.com> wrote:
Show 35 quoted lines
> > Ayush Chandekar <ayu.chandekar@gmail.com> writes: > > > On Wed, Jul 16, 2025 at 2:53 AM Junio C Hamano <gitster@pobox.com> wrote: > >> > > [snip] > >> > >> This patch is exactly what Phillip suggested in > >> > >> https://lore.kernel.org/git/9e96aaab-79a2-4632-94cd-d016d4a63b30@gmail.com/ > >> > >> isn't it? Makes sense to me. > >> > > > > Yes, you're right. I should add the suggested-by trailer for this patch. > > I am not sure about that, though. A verbatim copy is stronger than > implementing what was suggested by another person. If I were in > your position, I'll probably say something like > > The patch text was taken from Phillip Wood's message [*URL*], > with the commit log message written by me. > > Based-on-a-patch-by: Phillip Wood <...> > Signed-off-by: Ayush Chandekar <...> > > In any case, this overlaps both textually but also intent-wise with > Phillip's "let's mark core.commentchar=auto deprecated and remove > the support at 3.0 boundary", which is planned to be rerolled to > make it a failure when the user uses core.commentchar=auto. It > would be a while before we tag Git 3.0, so the fix in this topic > will be necessary until then. > > Thanks. >
Yeah, Phillip should actually get the primary credit for this patch and Suggested-by does not do enough justice. I will send a new version right away.
Thanks! Ayush