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

Re: git-archive and tar options

From
Jakub Narebski <jnareb@gmail.com>
Date
Jul 14, 2011, 19:12 UTC
Message-ID
<m3oc0wad94.fsf@localhost.localdomain>
In-Reply-To
<20110714181858.GA25172@sigill.intra.peff.net>
Jeff King <peff@peff.net> writes:
Show 26 quoted lines
> On Thu, Jul 14, 2011 at 07:45:07PM +0200, René Scharfe wrote:
> 
> > > We didn't when git-archive was written, but these days we have
> > > get_sha1_with_context to remember incidental things about an object we
> > > look up. It should perhaps remember the commit (if any) we used to reach
> > > a treeish, and then the above command line could still insert the pax
> > > header.
> > 
> > That's a good idea to increase consistency, as there shouldn't really be
> > a difference in output between the two subdirectory syntaxes.
> 
> The patch to do this is pretty tiny. See below.
> 
> There are a few issues, though:
> 
>   1. I think this is probably the right thing to do, and most people
>      will be happy about it. But I guess I can see an argument that the
>      commit-id should not be there, as the subtree does not represent
>      that commit.
> 
>      IOW, if you assume the commit-id in the output means
>      "by the way, this came from commit X", this change is a good thing.
>      If you assume it means "this is the tree from commit X", then it's
>      not.  I have no idea how people use it. I never have, but I always
>      assumed the use case was "I have this random tarball. Where did it
>      come from?".

Perhaps we should embed '<commit-id>:<subtree>' instead in pax header, in that case? Or <commit-id>.<subtree> if ':' is forbidden.

-- 
Jakub Narębski
Poland
Previous: Jeff KingNext: Junio C Hamano
Message 7 of 22 in “git-archive and tar options”
  1. Neal KreitzingerJul 13, 2011
  2. Jeff KingJul 14, 2011
  3. René ScharfeJul 14, 2011
  4. Jeff KingJul 14, 2011
  5. René ScharfeJul 14, 2011
  6. Jeff KingJul 14, 2011
  7. Jakub NarebskiJul 14, 2011
  8. Junio C HamanoJul 14, 2011
  9. Jeff KingJul 14, 2011
  10. Junio C HamanoJul 14, 2011
  11. René ScharfeJul 15, 2011
  12. Neal KreitzingerJul 18, 2011
  13. René ScharfeJul 18, 2011
  14. Jakub NarebskiJul 14, 2011
  15. Neal KreitzingerJul 18, 2011
  16. René ScharfeJul 18, 2011
  17. Neal KreitzingerJul 19, 2011
  18. René ScharfeJul 19, 2011
  19. Neal KreitzingerJul 21, 2011
  20. Neal KreitzingerJul 21, 2011
  21. Andreas SchwabJul 14, 2011
  22. Sylvain RabotJul 19, 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.