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

Re: [PATCHv5 2/4] Add infrastructure for ref namespaces

From
Josh Triplett <josh@joshtriplett.org>
Date
Jun 3, 2011, 23:43 UTC
Message-ID
<20110603234357.GA2170@leaf>
In-Reply-To
<7vd3iua4bm.fsf@alter.siamese.dyndns.org>
On Fri, Jun 03, 2011 at 04:22:05PM -0700, Junio C Hamano wrote:
Show 11 quoted lines
> Jamey Sharp <jamey@minilop.net> writes:
> 
> > Note that namespaces which include a / will expand to a hierarchy of
> > namespaces; for example, GIT_NAMESPACE=foo/bar will store refs under
> > refs/namespaces/foo/refs/namespaces/bar/.  This makes GIT_NAMESPACE
> > behave hierarchically, and avoids ambiguity with namespaces such as
> > foo/refs/heads.
> 
> I would have expected that this explanation would be improved after you
> having to answer my question on the list.  Otherwise it was a wasted
> effort, for both me (asking) and you (answering).

Just missed it when re-spinning the series. Will re-send with an updated commit message for that patch.

Show 8 quoted lines
> > This adds the infrastructure for ref namespaces: handling the
> > GIT_NAMESPACE environment variable and --namespace option, and iterating
> > over refs in a namespace.  Subsequent commits use this infrastructure to
> > implement the user-visible support for ref namespaces.
> >
> > Commit by Josh Triplett and Jamey Sharp.
> 
> Also please drop that "Commit by ...".  We can read Sign-off.

Signed-off-by doesn't quite have the right meaning, since normally a chain of signoffs just indicates the chain of review, not necessarily authorship. We brought this up at one point previously[1], and Jonathan Nieder pointed to previous discussions about this, with the apparent conclusion to use another pseudo-header to indicate co-authorship. (Also why we originally put it in the same paragraph as the signed-off-by.) Documentation/SubmittingPatches also mentions the possibility of adding extra pseudo-headers similar to Signed-off-by, as appropriate.

We really did write these patches (and commit messages) sitting right next to each other and switching off with the same keyboard. :)

- Josh Triplett and Jamey Sharp
[1] http://bugs.debian.org/451880
Previous: Junio C HamanoNext: Jamey Sharp
Message 5 of 9 in “[PATCHv5 0/4] Support ref namespaces”
  1. Jamey SharpJun 3, 2011
  2. 1/4 Fix prefix handling in ref iteration functionsJamey Sharp, Jun 3, 2011
  3. 2/4 Add infrastructure for ref namespacesJamey Sharp, Jun 3, 2011
  4. Junio C HamanoJun 3, 2011
  5. Josh TriplettJun 3, 2011
  6. 3/4 Support ref namespaces for remote repositories via upload-pack and receive-packJamey Sharp, Jun 3, 2011
  7. 4/4 Add documentation for ref namespacesJamey Sharp, Jun 3, 2011
  8. Junio C HamanoJun 3, 2011
  9. Josh TriplettJun 3, 2011

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.