{"thread":{"id":"12937","subject":"\"make test\" fails if /path/to/git.git contains spaces","startedAt":"2008-03-31T21:51:23Z","lastAt":"2008-04-03T02:26:18Z","messageCount":4,"participants":["Adam Roben","Bryan Donlan","Johannes Sixt"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"73459","messageId":"47F15CDB.60109@apple.com","threadId":"12937","inReplyTo":null,"subject":"\"make test\" fails if /path/to/git.git contains spaces","fromName":"Adam Roben","fromEmail":"aroben@apple.com","sentAt":"2008-03-31T21:51:23Z","receivedAt":"2008-03-31T21:51:23Z","isPatch":false,"sender":{"key":"aroben@apple.com","avatar":"https://gravatar.com/avatar/9d3697e1de53890adf241331f4b970bdd2b18962b2ff0b8028ebb00e085807f8?d=mp&s=160"},"body":"I've noticed that \"make test\" fails if /path/to/git.git contains spaces, \nwith the following error:\n\n> $ make test\n> make -C t/ all\n> make[1]: Entering directory `/home/Adam Roben/dev/git/t'\n> *** t0000-basic.sh ***\n> * error: cannot run git init -- have you built things yet?\n> make[1]: *** [t0000-basic.sh] Error 1\n> make[1]: Leaving directory `/home/Adam Roben/dev/git/t'\n> make: *** [test] Error 2\n\nThis is a very common configuration on Cygwin, but clearly not a common \nconfiguration for git developers, or else this error would have been \ncorrected already.\n\nDo we want to support having spaces in your path? It doesn't seem hard \nto fix, but it does seem like the kind of problem that will come up over \nand over again if only a very small set of people have this configuration.\n\n-Adam\n"},{"id":"73461","messageId":"3e8340490803311604v52ab9e03nd101ccadd4973760@mail.gmail.com","threadId":"12937","inReplyTo":"47F15CDB.60109@apple.com","subject":"Re: \"make test\" fails if /path/to/git.git contains spaces","fromName":"Bryan Donlan","fromEmail":"bdonlan@gmail.com","sentAt":"2008-03-31T23:04:05Z","receivedAt":"2008-03-31T23:04:05Z","isPatch":false,"sender":{"key":"bdonlan@gmail.com","avatar":null},"body":"On Mon, Mar 31, 2008 at 5:51 PM, Adam Roben wrote:\n> I've noticed that \"make test\" fails if /path/to/git.git contains spaces,\n> with the following error:\n>\n> > $ make test\n> > make -C t/ all\n> > make[1]: Entering directory `/home/Adam Roben/dev/git/t'\n> > *** t0000-basic.sh ***\n> > * error: cannot run git init -- have you built things yet?\n> > make[1]: *** [t0000-basic.sh] Error 1\n> > make[1]: Leaving directory `/home/Adam Roben/dev/git/t'\n> > make: *** [test] Error 2\n>\n> This is a very common configuration on Cygwin, but clearly not a common\n> configuration for git developers, or else this error would have been\n> corrected already.\n>\n> Do we want to support having spaces in your path? It doesn't seem hard\n> to fix, but it does seem like the kind of problem that will come up over\n> and over again if only a very small set of people have this configuration.\n\nTurns out that while trying to fix the tests, I've found that\ngit-rebase doesn't like a $VISUAL having spaces, so perhaps this\nshould be tested more often :) Or should we just require $VISUAL have\nno spaces?\n\nI'll send a patch once I've got everything passing.\n\nThanks,\n\nBryan\n"},{"id":"73486","messageId":"47F1D666.9060402@viscovery.net","threadId":"12937","inReplyTo":"3e8340490803311604v52ab9e03nd101ccadd4973760@mail.gmail.com","subject":"Re: \"make test\" fails if /path/to/git.git contains spaces","fromName":"Johannes Sixt","fromEmail":"j.sixt@viscovery.net","sentAt":"2008-04-01T06:29:58Z","receivedAt":"2008-04-01T06:29:58Z","isPatch":false,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"Bryan Donlan schrieb:\n> On Mon, Mar 31, 2008 at 5:51 PM, Adam Roben wrote:\n>> Do we want to support having spaces in your path? It doesn't seem hard\n>> to fix, but it does seem like the kind of problem that will come up over\n>> and over again if only a very small set of people have this configuration.\n> \n> Turns out that while trying to fix the tests, I've found that\n> git-rebase doesn't like a $VISUAL having spaces, so perhaps this\n> should be tested more often :) Or should we just require $VISUAL have\n> no spaces?\n> \n> I'll send a patch once I've got everything passing.\n\nThere was an attempt in the past to make this working. Please look at this\n(rather lengthy) thread, in particular, my critique on the patches that\nwere submitted:\n\nhttp://thread.gmane.org/gmane.comp.version-control.git/60544\n\nI really would not like to see the same issues that I took the time to\ncomment on to show up again.\n\n-- Hannes\n"},{"id":"73573","messageId":"3e8340490804021926pa0dc516r85c41c068aea01a8@mail.gmail.com","threadId":"12937","inReplyTo":"47F1D666.9060402@viscovery.net","subject":"Re: \"make test\" fails if /path/to/git.git contains spaces","fromName":"Bryan Donlan","fromEmail":"bdonlan@gmail.com","sentAt":"2008-04-03T02:26:18Z","receivedAt":"2008-04-03T02:26:18Z","isPatch":false,"sender":{"key":"bdonlan@gmail.com","avatar":null},"body":"On Tue, Apr 1, 2008 at 2:29 AM, Johannes Sixt wrote:\n> Bryan Donlan schrieb:\n>\n> > On Mon, Mar 31, 2008 at 5:51 PM, Adam Roben wrote:\n>\n> >> Do we want to support having spaces in your path? It doesn't seem hard\n> >> to fix, but it does seem like the kind of problem that will come up over\n> >> and over again if only a very small set of people have this configuration.\n> >\n> > Turns out that while trying to fix the tests, I've found that\n> > git-rebase doesn't like a $VISUAL having spaces, so perhaps this\n> > should be tested more often :) Or should we just require $VISUAL have\n> > no spaces?\n> >\n> > I'll send a patch once I've got everything passing.\n>\n> There was an attempt in the past to make this working. Please look at this\n> (rather lengthy) thread, in particular, my critique on the patches that\n> were submitted:\n>\n> http://thread.gmane.org/gmane.comp.version-control.git/60544\n>\n> I really would not like to see the same issues that I took the time to\n> comment on to show up again.\n\nThanks for the advice, I'll try not to make the same mistakes :)\nI've found a few legitimate bugs as I've been updating the tests, so\nI'd call this a worthwhile endeavour. I'll send the fixes in question\nas broken-out patches in the final patch series.\n\nEven if tests with space-ridden paths aren't run /often/, it'd still\nbe a good idea to fix them when they're noticed as broken, IMO. (I'm\ncurrently testing with quotes and backslashes, too, now :)\n\nThanks,\n\nBryan\n"}]}