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

Re: [PATCH] Provide config option to expect files outside sparse patterns

From
Derrick Stolee <derrickstolee@github.com>
Date
Feb 22, 2022, 13:43 UTC
Message-ID
<f65462bf-eed7-f8b9-6255-249e5128172f@github.com>
In-Reply-To
<nycvar.QRO.7.76.6.2202221313400.11118@tvgsbejvaqbjf.bet>
On 2/22/2022 7:28 AM, Johannes Schindelin wrote:
Show 26 quoted lines
> Hi Elijah,
> 
> On Mon, 21 Feb 2022, Elijah Newren wrote:
> 
>> On Sun, Feb 20, 2022 at 11:41 AM Derrick Stolee
>> <derrickstolee@github.com> wrote:
>>
>>> The only concern here really is if we want to be picky about the "VFS
>>> for Git" references instead of "vfsd" references in the commit
>>> message.
>>
>> I'm not sure I'm understanding the distinction.  Was "VFS for Git"
>> renamed to "vfsd"?  Is "vfsd" Google's modified version?  Something
>> else?
> 
> VFS for Git is the existing project at
> https://github.com/microsoft/VFSforGit which is pretty much in maintenance
> mode.
> 
> vfsd was mentioned by Jonathan Tan in
> https://lore.kernel.org/git/20220207190320.2960362-1-jonathantanmy@google.com/.
> From what I gather, it is a completely separate implementation of the same
> ideas of VFS for Git, but from what I see it does not share any code with
> it (and it is unclear how much vfsd tries/tried to learn from VFS for Git,
> it looks like it's being done from scratch but that impression could be
> incorrect).

Elijah is correct that "vfsd" is not publicly visible outside of this message, so that makes it difficult for us to verify that these patches being made for it actually work as we intend.

> 	Side note: VFS for Git itself requires the Microsoft fork of Git
> 	to work, therefore this patch would only be needed in that fork,
> 	as far as VFS for Git is concerned.

I would drop this side note, since a big reason for the microsoft/git fork is so we can create custom patches that handle these issues. We would probably have created a similar patch during release integration, but we are grateful to use this patch as a base for making these integrations simpler in the future.

Thanks, -Stolee

Previous: Johannes SchindelinNext: Johannes Schindelin
Message 6 of 29 in “Provide config option to expect files outside sparse patterns”
  1. Provide config option to expect files outside sparse patternsElijah Newren via GitGitGadget, Feb 20, 2022
  2. Derrick StoleeFeb 20, 2022
  3. Junio C HamanoFeb 20, 2022
  4. Elijah NewrenFeb 22, 2022
  5. Johannes SchindelinFeb 22, 2022
  6. Derrick StoleeFeb 22, 2022
  7. Johannes SchindelinFeb 21, 2022
  8. Ævar Arnfjörð BjarmasonFeb 21, 2022
  9. Elijah NewrenFeb 22, 2022
  10. Johannes SchindelinFeb 22, 2022
  11. Ævar Arnfjörð BjarmasonFeb 22, 2022
  12. Jonathan NiederFeb 22, 2022
  13. Ævar Arnfjörð BjarmasonFeb 25, 2022
  14. Elijah NewrenFeb 22, 2022
  15. Ævar Arnfjörð BjarmasonFeb 22, 2022
  16. Johannes SchindelinFeb 22, 2022
  17. Derrick StoleeFeb 22, 2022
  18. repo_read_index: add config to expect files outside sparse patternsJonathan Nieder, Feb 23, 2022
  19. Elijah NewrenFeb 23, 2022
  20. repo_read_index: add config to expect files outside sparse patternsElijah Newren, Feb 24, 2022
  21. Junio C HamanoFeb 24, 2022
  22. Elijah NewrenFeb 26, 2022
  23. Jonathan NiederFeb 25, 2022
  24. Elijah NewrenFeb 26, 2022
  25. repo_read_index: add config to expect files outside sparse patternsElijah Newren, Feb 26, 2022
  26. repo_read_index: add config to expect files outside sparse patternsElijah Newren, Mar 2, 2022
  27. Junio C HamanoMar 2, 2022
  28. Elijah NewrenMar 2, 2022
  29. Derrick StoleeMar 2, 2022

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.