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

Re: [PATCH v4 1/3] test: use unambigous leading path (/foo) for mingw

From
Sebastian Schuberth <sschuberth@gmail.com>
Date
Oct 10, 2013, 20:32 UTC
Message-ID
<52570EC8.3050207@gmail.com>
In-Reply-To
<15724e27cbef5067902ebebb93aa77d3605402c8.1379644482.git.worldhello.net@gmail.com>
On 20.09.2013 04:38, Jiang Xin wrote:
> In test cases for relative_path, path with one leading character
> (such as /a, /x) may be recogonized as "a:/" or "x:/" if there is
> such DOS drive on MINGW platform. Use an umambigous leading path
> "/foo" instead.
 >
 > Also change two leading slashes (//) to three leading slashes (///),
 > otherwize it will be recognized as UNC name on MINGW platform.

Note that the path mangling comes from MSYS [1], not MinGW, so you should place "MINGW" with "MSYS" in several places. As a side-note, the official spelling is "MinGW", not "MINGW".

> -relative_path /a/b/c/	/a/b/		c/
> +relative_path /foo/a/b/c/	/foo/a/b/	c/

Wouldn't it have been more straight-forward to just replace "a" with "foo", "b" with "bar" and "c" with "baz" (or whatever)? So that the first line would say

relative_path /foo/bar/baz/	/foo/bar/		baz/
Thanks for the fix!
[1] http://www.mingw.org/wiki/Posix_path_conversion
-- 
Sebastian Schuberth
Previous: Jiang XinNext: Jiang Xin
Message 35 of 48 in “Regression in e02ca72: git svn rebase is broken on Windows”
  1. TvangesteSep 10, 2013
  2. Johannes SchindelinSep 10, 2013
  3. TvangesteSep 10, 2013
  4. Johannes SchindelinSep 10, 2013
  5. TvangesteSep 10, 2013
  6. Johannes SixtSep 10, 2013
  7. TvangesteSep 10, 2013
  8. Junio C HamanoSep 10, 2013
  9. Karsten BleesSep 10, 2013
  10. Jiang XinSep 11, 2013
  11. Jiang XinSep 11, 2013
  12. Johannes SixtSep 11, 2013
  13. 1/2 relative_path should honor dos_drive_prefixJiang Xin, Sep 12, 2013
  14. 2/2 Use simpler relative_path when set_git_dirJiang Xin, Sep 12, 2013
  15. Johannes SixtSep 12, 2013
  16. TvangesteSep 12, 2013
  17. Torsten BögershausenSep 12, 2013
  18. Junio C HamanoSep 12, 2013
  19. Johannes SixtSep 12, 2013
  20. Junio C HamanoSep 12, 2013
  21. Jiang XinSep 13, 2013
  22. Junio C HamanoSep 13, 2013
  23. Jiang XinSep 17, 2013
  24. 1/3 test: use unambigous leading path (/foo) for mingwJiang Xin, Sep 17, 2013
  25. 2/3 relative_path should honor DOS and UNC pathsJiang Xin, Sep 17, 2013
  26. Junio C HamanoSep 17, 2013
  27. Jiang XinSep 18, 2013
  28. Junio C HamanoSep 18, 2013
  29. 3/3 Use simpler relative_path when set_git_dirJiang Xin, Sep 17, 2013
  30. Johannes SixtSep 17, 2013
  31. Torsten BögershausenSep 18, 2013
  32. Jiang XinSep 20, 2013
  33. 0/3 relative path regression fixJiang Xin, Sep 20, 2013
  34. 1/3 test: use unambigous leading path (/foo) for mingwJiang Xin, Sep 20, 2013
  35. Sebastian SchuberthOct 10, 2013
  36. Jiang XinOct 14, 2013
  37. 0/3 relative path regression fixJiang Xin, Oct 14, 2013
  38. 1/3 test: use unambigous leading path (/foo) for MSYSJiang Xin, Oct 14, 2013
  39. Sebastian SchuberthOct 14, 2013
  40. Eric SunshineOct 14, 2013
  41. 2/3 relative_path should honor dos-drive-prefixJiang Xin, Oct 14, 2013
  42. 3/3 Use simpler relative_path when set_git_dirJiang Xin, Oct 14, 2013
  43. 2/3 relative_path should honor dos-driver-prefixJiang Xin, Sep 20, 2013
  44. Sebastian SchuberthOct 10, 2013
  45. 3/3 Use simpler relative_path when set_git_dirJiang Xin, Sep 20, 2013
  46. Torsten BögershausenSep 13, 2013
  47. Karsten BleesSep 12, 2013
  48. Torsten BögershausenSep 18, 2013

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.