git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 2/2] t5556: replace test_i18ngrep with a simple grep

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 28, 2018, 16:18 UTC
Message-ID
<xmqqo9k9m6jm.fsf@gitster-ct.c.googlers.com>
In-Reply-To
<ffc3d3dc-6d21-4273-5362-cbf32e7e4e59@ramsayjones.plus.com>
Ramsay Jones <ramsay@ramsayjones.plus.com> writes:
Show 9 quoted lines
>> However, I'm not sure about the authorship and taking credit for the
>> patch.  We ended up taking my patch, sure, but I think Ramsay did all
>> the real hard work, i.e. writing the commit message and, most
>> importantly, realizing that something is wrong with that '...| sort' at
>> the end of the line.
>
> No, the patch and the credit are yours, I was just trying to
> help out and get the patches moving forward. At most, I would
> think a 'Helped-by:' would be sufficient to note my input.
OK.
> [BTW, my 'Signed-off-by:' on that patch was in the spirit of
> the dco section b. - again I wasn't quite sure ...]

I take your "wasn't" to imply that by now you are a bit more sure? It was a perfectly fine "I am passing it along" sign-off.

Thanks, both.
Previous: Ramsay Jones
Message 15 of 15 in “t5556: replace test_i18ngrep with a simple grep”
  1. 2/2 t5556: replace test_i18ngrep with a simple grepRamsay Jones, Feb 12, 2018
  2. Junio C HamanoFeb 12, 2018
  3. Junio C HamanoFeb 12, 2018
  4. Ramsay JonesFeb 13, 2018
  5. Junio C HamanoFeb 13, 2018
  6. SZEDER GáborFeb 13, 2018
  7. Junio C HamanoFeb 13, 2018
  8. Jeff KingFeb 13, 2018
  9. Junio C HamanoFeb 13, 2018
  10. Junio C HamanoFeb 27, 2018
  11. Junio C HamanoFeb 27, 2018
  12. Ramsay JonesFeb 27, 2018
  13. SZEDER GáborFeb 28, 2018
  14. Ramsay JonesFeb 28, 2018
  15. Junio C HamanoFeb 28, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.