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

Re: [PATCH v5 0/1] upload-pack.c: fix filter spec quoting bug

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 2, 2021, 05:49 UTC
Message-ID
<xmqq8s87hvd7.fsf@gitster.c.googlers.com>
In-Reply-To
<20210201202938.39576-1-jacob@gitlab.com>
Jacob Vosmaer <jacob@gitlab.com> writes:
> I updated the test based on the latest round of feedback.

Thanks, but this came a bit too late to wholesale replace the patch; the previous round was good enough and is in 'next' already.

Looking at the change between two versions, I guess that the change in tests may be worth salvaging, so perhaps you can make it into an incremental "the test we added earlier was good, but let's make it even better by doing X and Y" patch to be applied on top?

Thanks.
Show 6 quoted lines
> Jacob Vosmaer (1):
>   upload-pack.c: fix filter spec quoting bug
>
>  t/t5544-pack-objects-hook.sh | 10 ++++++++++
>  upload-pack.c                |  9 +--------
>  2 files changed, 11 insertions(+), 8 deletions(-)
Previous: Jacob VosmaerNext: Jacob Vosmaer
Message 20 of 24 in “Re: [PATCH v2] upload-pack.c: fix filter spec quoting bug”
  1. Jeff KingJan 25, 2021
  2. 0/1 upload-pack.c: fix filter spec quoting bugJacob Vosmaer, Jan 25, 2021
  3. 1/1 upload-pack.c: fix filter spec quoting bugJacob Vosmaer, Jan 25, 2021
  4. Ævar Arnfjörð BjarmasonJan 26, 2021
  5. Jacob VosmaerJan 26, 2021
  6. Junio C HamanoJan 26, 2021
  7. Jeff KingJan 26, 2021
  8. upload-pack.c: fix filter spec quoting bugJacob Vosmaer, Jan 28, 2021
  9. Junio C HamanoJan 26, 2021
  10. Jeff KingJan 26, 2021
  11. Junio C HamanoJan 26, 2021
  12. Junio C HamanoJan 26, 2021
  13. Jacob VosmaerJan 28, 2021
  14. Jacob VosmaerJan 28, 2021
  15. Jeff KingJan 28, 2021
  16. Jacob VosmaerFeb 1, 2021
  17. Junio C HamanoJan 28, 2021
  18. 0/1 upload-pack.c: fix filter spec quoting bugJacob Vosmaer, Feb 1, 2021
  19. 1/1 upload-pack.c: fix filter spec quoting bugJacob Vosmaer, Feb 1, 2021
  20. Junio C HamanoFeb 2, 2021
  21. 1/1 t5544: clarify 'hook works with partial clone' testJacob Vosmaer, Feb 2, 2021
  22. Eric SunshineFeb 2, 2021
  23. t5544: clarify 'hook works with partial clone' testJacob Vosmaer, Feb 2, 2021
  24. Junio C HamanoFeb 2, 2021

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.