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

Re: [PATCH 2/3] Quoting paths in tests

From
Johannes Sixt <j.sixt@viscovery.net>
Date
Oct 15, 2007, 13:47 UTC
Message-ID
<47136F71.1050107@viscovery.net>
In-Reply-To
<11924540291536-git-send-email-maillist@steelskies.com>
Jonathan del Strother schrieb:
Show 6 quoted lines
> -	svn import -m 'import for git-svn' . $svnrepo >/dev/null &&
> +	svn import -m 'import for git-svn' . '$svnrepo' >/dev/null &&
>  	cd .. &&
>  	rm -rf import &&
> -	git-svn init $svnrepo"
> +	git-svn init '$svnrepo'"

I don't see the point in changing an incorrect quoting to a different incorrect quoting that you fix up in a follow-up patch. It's *two* large patches to review instead of just one. I'm stopping the review here.

-- Hannes
Previous: Jonathan del StrotherNext: Jonathan del Strother
Message 34 of 38 in “Fixing path quoting issues”
  1. Fixing path quoting issuesJonathan del Strother, Oct 10, 2007
  2. Johannes SixtOct 11, 2007
  3. David KastrupOct 11, 2007
  4. Johannes SixtOct 11, 2007
  5. Jonathan del StrotherOct 11, 2007
  6. Johannes SixtOct 11, 2007
  7. David KastrupOct 11, 2007
  8. Jonathan del StrotherOct 11, 2007
  9. Johannes SchindelinOct 11, 2007
  10. David KastrupOct 11, 2007
  11. Johannes SixtOct 12, 2007
  12. Wincent ColaiutaOct 12, 2007
  13. Johannes SchindelinOct 12, 2007
  14. Wincent ColaiutaOct 12, 2007
  15. Johannes SchindelinOct 12, 2007
  16. David KastrupOct 12, 2007
  17. Jonathan del StrotherOct 13, 2007
  18. Andreas EricssonOct 13, 2007
  19. Jonathan del StrotherOct 15, 2007
  20. 1/3 Fixing path quoting in git-rebaseJonathan del Strother, Oct 15, 2007
  21. Johannes SixtOct 15, 2007
  22. Jonathan del StrotherOct 17, 2007
  23. Quoting paths, take 3Jonathan del Strother, Oct 17, 2007
  24. 1/2 Fixing path quoting in git-rebaseJonathan del Strother, Oct 17, 2007
  25. 2/2 Quoting paths in testsJonathan del Strother, Oct 17, 2007
  26. Johannes SixtOct 17, 2007
  27. Jonathan del StrotherOct 17, 2007
  28. David KastrupOct 17, 2007
  29. David KastrupOct 17, 2007
  30. Johannes SixtOct 18, 2007
  31. Jonathan del StrotherOct 24, 2007
  32. Johannes SixtOct 17, 2007
  33. 2/3 Quoting paths in testsJonathan del Strother, Oct 15, 2007
  34. Johannes SixtOct 15, 2007
  35. Jonathan del StrotherOct 15, 2007
  36. Johannes SixtOct 15, 2007
  37. David KastrupOct 15, 2007
  38. 3/3 Fix apostrophe quoting in testsJonathan del Strother, Oct 15, 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.