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

Re: [PATCH] Allow fetching references from any namespace

From
Alex Riesen <raa.lkml@gmail.com>
Date
May 12, 2007, 07:40 UTC
Message-ID
<20070512074049.GA2410@steel.home>
In-Reply-To
<7vlkfv9j3c.fsf@assigned-by-dhcp.cox.net>
Junio C Hamano, Fri, May 11, 2007 22:54:47 +0200:
Show 16 quoted lines
> Alex Riesen <raa.lkml@gmail.com> writes:
> 
> > not only from the three defined: heads, tags and remotes.
> >
> > Noticed when I tried to fetch the references created by git-p4-import.bat:
> > they are placed into separate namespace (refs/p4import/, to avoid showing
> > them in git-branch output). As canon_refs_list_for_fetch always prepended
> > refs/heads/ it was impossible, and annoying: it worked before. Normally,
> > the p4import references are useless anywhere but in the directory managed
> > by perforce, but in this special case the cloned directory was supposed
> > to be a backup, including the p4import branch: it keeps information about
> > where the imported perforce state came from.
> 
> Have no objection to the patch itself, but mind pointing out
> where we broke it (I suspect it is around 1.5.0)?
> 
Maybe even much earlier. According to
    git log -p --decorate -- git-fetch-script git-parse-remote-script
    commit ac4b0cff00b7629657e61a1d6e1f1a1250d03198
    Author: Junio C Hamano <junkio@cox.net>
    Date:   Sat Aug 20 02:52:24 2005 -0700
	[PATCH] Start adding the $GIT_DIR/remotes/ support.
    +canon_refs_list_for_fetch () {
    +       for ref
    +       do
    +               expr "$ref" : '.*:' >/dev/null || ref="${ref}:"
    +               remote=$(expr "$ref" : '\([^:]*\):')
    +               local=$(expr "$ref" : '[^:]*:\(.*\)')
    +               case "$remote" in
    +               '') remote=HEAD ;;
    +               *) remote="refs/heads/$remote" ;;
    +               esac
    +               case "$local" in
    +               '') local= ;;
    +               *) local="refs/heads/$local" ;;
    +               esac
    +               echo "${remote}:${local}"
    +       done
    +}
it was this way since at least Aug 2005.
Previous: Junio C HamanoNext: Junio C Hamano
Message 3 of 6 in “Allow fetching references from any namespace”
  1. Allow fetching references from any namespaceAlex Riesen, May 11, 2007
  2. Junio C HamanoMay 11, 2007
  3. Alex RiesenMay 12, 2007
  4. Junio C HamanoMay 12, 2007
  5. Marco CostalbaMay 13, 2007
  6. Jan HudecMay 13, 2007

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.