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

Re: Memory overrun in http-push.c

From
Daniel Barkalow <barkalow@iabervon.org>
Date
Mar 4, 2007, 08:17 UTC
Message-ID
<Pine.LNX.4.64.0703040234510.6485@iabervon.org>
In-Reply-To
<200703021005.13620.andyparkins@gmail.com>
On Fri, 2 Mar 2007, Andy Parkins wrote:
Show 10 quoted lines
> What about this though:
>  * Tag a release
>  * Export the working tree into a fresh directory
>  * Edit each source file to put the hash of the tagged revision into
>    every file.
>  * Edit the makefile to include the tag hash in the released version
>  * tar it up
>  * Release
> It's such a mundane, useless waste of time to edit the hash in by hand - why 
> can't the version control system do it?

Actually, how about doing keyword expansion just like the new CRLF handling? Have the text filter replace $Id...$ with $Id$ going into the database, and $Id$ with $Id <info>$ coming out of the database. The info could even include the commit (although that might be confusing for files that you haven't changed since you checked them out, and when you checked them out, they weren't changed in that commit; the commit would be a sort of arbitrary choice among commits that have that file).

It seems to me like keywords fit the pattern we now have (with CRLF-conversion) of data which isn't tracked, but is present in working directory files. In the CRLF case, it's a constant character (CR), but there's no reason we couldn't have the non-canonical form depend on more information. (And we'd beat CVS/SVN for accuracy: if keyword substitution isn't on, nothing appears in $Id$ tags, rather than being left with something inaccurate).

	-Daniel
*This .sig left intentionally blank*
Previous: Andy ParkinsNext: Junio C Hamano
Message 25 of 39 in “Memory overrun in http-push.c”
  1. Eygene RyabinkinFeb 28, 2007
  2. Andy ParkinsFeb 28, 2007
  3. Johannes SchindelinFeb 28, 2007
  4. Eygene RyabinkinMar 1, 2007
  5. Alex RiesenMar 1, 2007
  6. Eygene RyabinkinMar 1, 2007
  7. Alex RiesenMar 1, 2007
  8. Eygene RyabinkinMar 1, 2007
  9. Junio C HamanoMar 1, 2007
  10. Alex RiesenMar 1, 2007
  11. Andy ParkinsMar 1, 2007
  12. Eygene RyabinkinMar 1, 2007
  13. Junio C HamanoMar 1, 2007
  14. Eygene RyabinkinMar 1, 2007
  15. Johannes SchindelinMar 1, 2007
  16. Andy ParkinsMar 1, 2007
  17. Johannes SchindelinMar 1, 2007
  18. Andy ParkinsMar 1, 2007
  19. Johannes SchindelinMar 1, 2007
  20. Andy ParkinsMar 2, 2007
  21. Jakub NarebskiMar 2, 2007
  22. Andy ParkinsMar 2, 2007
  23. Johannes SchindelinMar 2, 2007
  24. Andy ParkinsMar 2, 2007
  25. Daniel BarkalowMar 4, 2007
  26. Junio C HamanoMar 4, 2007
  27. Daniel BarkalowMar 4, 2007
  28. Alex RiesenMar 1, 2007
  29. Shawn O. PearceMar 1, 2007
  30. Uwe Kleine-KönigMar 1, 2007
  31. Jakub NarebskiMar 2, 2007
  32. Johannes SchindelinMar 2, 2007
  33. identifying blobs (was Re: Memory overrun in http-push.c)Junio C Hamano, Mar 2, 2007
  34. Linus TorvaldsMar 2, 2007
  35. Andy ParkinsMar 2, 2007
  36. Matthieu MoyMar 2, 2007
  37. Andy ParkinsMar 2, 2007
  38. Florian WeimerFeb 28, 2007
  39. Eygene RyabinkinMar 1, 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.