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

Re: [PATCH] diff-raw format update take #2.

From
Linus Torvalds <torvalds@osdl.org>
Date
May 24, 2005, 01:03 UTC
Message-ID
<Pine.LNX.4.58.0505231758350.2307@ppc970.osdl.org>
In-Reply-To
<7v64x91mfb.fsf@assigned-by-dhcp.cox.net>
On Mon, 23 May 2005, Junio C Hamano wrote:
> 
> Embedded spaces in path is _always_ safe.

For raw-diff yes, but since you'd normally end up using that name in the diff, it won't be safe any more.

Imagine a name like "this is a file", and think about how the diff ends up looking:

	diff --git a/this is a file b/this is a file
and realize that that can't be parsed sanely by anybody who uses the diff.

And here '-z' doesn't help us, because we're basically screwed by the diff format (not our own decision).

So CVS uses "Index: " to help this somewhat, and we can get it right for renames and copies (because we then output the name in a way that is at least space and tab-safe, if not newline-safe). But basically, anything that uses patches as a medium for passing information around should _really_ avoid using spaces or tabs in filenames, and that's quite independent of git ;/

		Linus
Previous: Junio C HamanoNext: Chris Wedgwood
Message 12 of 20 in “Make sure diff-helper can tell rename/copy in the new diff-raw format.”
  1. Make sure diff-helper can tell rename/copy in the new diff-raw format.Junio C Hamano, May 23, 2005
  2. Linus TorvaldsMay 23, 2005
  3. Junio C HamanoMay 23, 2005
  4. Linus TorvaldsMay 23, 2005
  5. Linus TorvaldsMay 23, 2005
  6. diff-raw format update take #2.Junio C Hamano, May 23, 2005
  7. diff-raw format update take #2.Junio C Hamano, May 23, 2005
  8. Chris WedgwoodMay 24, 2005
  9. Junio C HamanoMay 24, 2005
  10. Chris WedgwoodMay 24, 2005
  11. Junio C HamanoMay 24, 2005
  12. Linus TorvaldsMay 24, 2005
  13. Chris WedgwoodMay 24, 2005
  14. David LangMay 24, 2005
  15. Chris WedgwoodMay 24, 2005
  16. Thomas GlanzmannMay 24, 2005
  17. Linus TorvaldsMay 24, 2005
  18. Linus TorvaldsMay 24, 2005
  19. Junio C HamanoMay 26, 2005
  20. Thomas GlanzmannMay 24, 2005

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.