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

Re: [PATCH v2] upload-pack: fix filter options scope

From
Christian Couder <christian.couder@gmail.com>
Date
May 8, 2020, 18:00 UTC
Message-ID
<CAP8UFD2-8FwEjyHromp-=MSSeFo=mn9GhkPvtFnF9MRx-4cjPQ@mail.gmail.com>
In-Reply-To
<20200508171635.GB637136@coredump.intra.peff.net>
On Fri, May 8, 2020 at 7:16 PM Jeff King <peff@peff.net> wrote:
>
> On Fri, May 08, 2020 at 08:46:58AM -0700, Junio C Hamano wrote:
Show 11 quoted lines
> > Is it that the server side was incapable of serving a follow-up
> > fetch request in the same process when protocol v2 was in use?
> > Perhaps
> >
> >     upload-pack: allow follow-up fetch in protocol v2
> >
> > or something?
>
> Yeah, I agree that's better. Maybe even:
>
>   upload-pack: clear filter_options for each v2 fetch command
Ok, I will resend soon with the above title.
> which is more direct (we already allow follow-up fetches; they just
> don't work sometimes) and makes it clear that this is about fixing the
> filter feature with v2.

Thanks, Christian.

Previous: Jeff KingNext: Junio C Hamano
Message 10 of 12 in “upload-pack: fix filter options scope”
  1. upload-pack: fix filter options scopeChristian Couder, May 7, 2020
  2. Jeff KingMay 7, 2020
  3. Christian CouderMay 7, 2020
  4. Jeff KingMay 7, 2020
  5. Taylor BlauMay 7, 2020
  6. upload-pack: fix filter options scopeChristian Couder, May 8, 2020
  7. Jeff KingMay 8, 2020
  8. Junio C HamanoMay 8, 2020
  9. Jeff KingMay 8, 2020
  10. Christian CouderMay 8, 2020
  11. Junio C HamanoMay 8, 2020
  12. upload-pack: clear filter_options for each v2 fetch commandChristian Couder, May 8, 2020

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.