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

Re: [PATCH] clarify documentation for remote helpers

From
Junio C Hamano <gitster@pobox.com>
Date
Aug 30, 2019, 17:34 UTC
Message-ID
<xmqqv9uey2tb.fsf@gitster-ct.c.googlers.com>
In-Reply-To
<88d9063060644dd99122bf0933f23fa9@exmbdft7.ad.twosigma.com>
David Turner <David.Turner@twosigma.com> writes:
Show 5 quoted lines
> I was confused, because I read "a file under GIT_DIR/objects/pack"
> to mean "just the filename".  Some of the things that deal with
> packs take just the filename (e.g. --keep-pack for git repack).
> I'll fix the under under and add $, but I do want to clarify that
> it's the full path.

I think that the phrase wanted to say that the file named with the option must live under that directory, without any implication that the directory is used as the base when a relative path is used. If the helper MUST give a full pathname and a pathname relative to that directory is not accepted, it is a good idea to spell it out (also if it must end with ".keep", that also should be documented---is there any other restrictions?).

Previous: David TurnerNext: David Turner
Message 4 of 6 in “clarify documentation for remote helpers”
  1. clarify documentation for remote helpersDavid Turner, Aug 29, 2019
  2. Martin ÅgrenAug 30, 2019
  3. David TurnerAug 30, 2019
  4. Junio C HamanoAug 30, 2019
  5. David TurnerAug 30, 2019
  6. Junio C HamanoAug 30, 2019

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.