Re: Bug: Recursive submodules fail when the repo path contains spaces

2012-07-29 Thread Phil Hord
On Tue, Jul 24, 2012 at 4:33 PM, Justin Spahr-Summers justin.spahrsumm...@gmail.com wrote: On Tuesday, 24. July 2012 at 13:26, Junio C Hamano wrote: Jens Lehmann jens.lehm...@web.de (http://web.de) writes: Am 24.07.2012 21:01, schrieb Justin Spahr-Summers: This occurs on Mac OS X 10.7.4,

Re: Bug: Recursive submodules fail when the repo path contains spaces

2012-07-29 Thread Justin Spahr-Summers
On Sunday, 29. July 2012 at 11:31, Phil Hord wrote: On Tue, Jul 24, 2012 at 4:33 PM, Justin Spahr-Summers justin.spahrsumm...@gmail.com (mailto:justin.spahrsumm...@gmail.com) wrote: On Tuesday, 24. July 2012 at 13:26, Junio C Hamano wrote: Jens Lehmann jens.lehm...@web.de (http://web.de)

Re: Bug: Recursive submodules fail when the repo path contains spaces

2012-07-29 Thread Phil Hord
On Sun, Jul 29, 2012 at 3:30 PM, Justin Spahr-Summers justin.spahrsumm...@gmail.com wrote: Hmm, you're right. After further investigation, this actually looks like a weird issue with soft links. Here's some exact output (with spacing only for clarity), using a public test repository at

Re: Bug: Recursive submodules fail when the repo path contains spaces

2012-07-29 Thread Justin Spahr-Summers
On Sunday, 29. July 2012 at 14:10, Phil Hord wrote: On Sun, Jul 29, 2012 at 3:30 PM, Justin Spahr-Summers justin.spahrsumm...@gmail.com (mailto:justin.spahrsumm...@gmail.com) wrote: Hmm, you're right. After further investigation, this actually looks like a weird issue with soft links.

Re: Bug: Recursive submodules fail when the repo path contains spaces

2012-07-28 Thread Heiko Voigt
On Tue, Jul 24, 2012 at 01:33:44PM -0700, Justin Spahr-Summers wrote: Here's some real output, with a couple specific names removed, starting from the root of the top-level repository (where External/twui is a submodule): $ cd External/twui $ git submodule add

Re: Bug: Recursive submodules fail when the repo path contains spaces

2012-07-28 Thread Justin Spahr-Summers
On Saturday, 28. July 2012 at 09:21, Heiko Voigt wrote: On Tue, Jul 24, 2012 at 01:33:44PM -0700, Justin Spahr-Summers wrote: Here's some real output, with a couple specific names removed, starting from the root of the top-level repository (where External/twui is a submodule): $ cd

Re: Bug: Recursive submodules fail when the repo path contains spaces

2012-07-24 Thread Jens Lehmann
Am 24.07.2012 21:01, schrieb Justin Spahr-Summers: This occurs on Mac OS X 10.7.4, on git versions 1.7.10.2 (Apple Git-33) and 1.7.11.3. Steps: 1. Create or clone a repository to an absolute path that contains spaces. 2. Add a submodule to the repository, if it does not already have one.

Re: Bug: Recursive submodules fail when the repo path contains spaces

2012-07-24 Thread Junio C Hamano
Jens Lehmann jens.lehm...@web.de writes: Am 24.07.2012 21:01, schrieb Justin Spahr-Summers: This occurs on Mac OS X 10.7.4, on git versions 1.7.10.2 (Apple Git-33) and 1.7.11.3. Steps: 1. Create or clone a repository to an absolute path that contains spaces. 2. Add a submodule to the

Re: Bug: Recursive submodules fail when the repo path contains spaces

2012-07-24 Thread Justin Spahr-Summers
On Tuesday, 24. July 2012 at 13:08, Jens Lehmann wrote: Am 24.07.2012 21:01, schrieb Justin Spahr-Summers: This occurs on Mac OS X 10.7.4, on git versions 1.7.10.2 (Apple Git-33) and 1.7.11.3. Steps: 1. Create or clone a repository to an absolute path that contains spaces. 2. Add a

Re: Bug: Recursive submodules fail when the repo path contains spaces

2012-07-24 Thread Justin Spahr-Summers
On Tuesday, 24. July 2012 at 13:26, Junio C Hamano wrote: Jens Lehmann jens.lehm...@web.de (http://web.de) writes: Am 24.07.2012 21:01, schrieb Justin Spahr-Summers: This occurs on Mac OS X 10.7.4, on git versions 1.7.10.2 (Apple Git-33) and 1.7.11.3. Steps: 1. Create or

Re: Bug: Recursive submodules fail when the repo path contains spaces

2012-07-24 Thread Junio C Hamano
Justin Spahr-Summers justin.spahrsumm...@gmail.com writes: On Tuesday, 24. July 2012 at 13:26, Junio C Hamano wrote: ... I can see one codepath that would behave incorrectly,... ... My recommendation at this point (i.e. not a long term) for people with problems Justin saw is Don't do it