Re: [PATCH] test-lib.sh - cygwin does not have usable FIFOs
On 07/13/2013 08:57 PM, Jonathan Nieder wrote: I'm not sure I follow. Are you saying Windows users would never want to access Subversion repositories? Thanks, Jonathan Quite the contrary. SVN and git both work on Windows without having POSIX FIFOs - Windows does have FIFOS, but the semantics are different than POSIX and this is why Cygwin's do not work as needed. There is no problem having a subprocess with stdin and stdout redirected via anonymous pipes to the parent: this is how git runs sub commands and works fine. I'm just questioning why this same construct cannot be used for the test harness. Mark -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] test-lib.sh - cygwin does not have usable FIFOs
Torsten Bögershausen wrote: > Disabling PIPE under cygwin seems to be the right thing to do, > or do I miss something ? If fifos don't work on Cygwin, disabling that test prerequisite is defintely the right thing to do. I was taking the opportunity to find out whether and how git could be tweaked to avoid needing fifos when setting pipes up in scripts. Jonathan -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] test-lib.sh - cygwin does not have usable FIFOs
Mark Levedahl wrote: > However, I don't understand why git would need to consume its own > output - If named pipes are really needed to use git-svn because > git-svn depends upon git feeding the same git process, then that > package should not be available on cygwin or any other platform that > does not support fifos. This isn't about git-svn but about the svn remote helper. The same considerations would apply to any other foreign repository importer that uses "git fast-import --cat-blob-fd". So I would like to make sure it is at least possible to support such a thing in the Cygwin and mingw ports. > If not, then I don't think the test suite > should require fifos or any other construct with the same git > process feeding itself either, it just blurs the line about what is > actually being tested. I'm not sure I follow. Are you saying Windows users would never want to access Subversion repositories? Thanks, Jonathan -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] test-lib.sh - cygwin does not have usable FIFOs
On 07/06/2013 08:55 PM, Jonathan Nieder wrote: Mark Levedahl wrote: Do not use FIFOs on cygwin, they do not work. Cygwin includes coreutils, so has mkfifo, and that command does something. However, the resultant named pipe is known (on the Cygwin mailing list at least) to not work correctly. Hm. How would you recommend going about writing a script that takes output from a command, transforms it, and then feeds it back into that command's input? Are sockets a more reliable way to do this kind of IPC on Cygwin? See reinit_git and try_dump from t9010-svn-fe.sh for context. Thanks, Jonathan On the one hand, sockets work fine on cygwin so that path would probably work. However, I don't understand why git would need to consume its own output - If named pipes are really needed to use git-svn because git-svn depends upon git feeding the same git process, then that package should not be available on cygwin or any other platform that does not support fifos. If not, then I don't think the test suite should require fifos or any other construct with the same git process feeding itself either, it just blurs the line about what is actually being tested. Mark -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] test-lib.sh - cygwin does not have usable FIFOs
On 2013-07-07 02.55, Jonathan Nieder wrote: > Mark Levedahl wrote: > >> Do not use FIFOs on cygwin, they do not work. Cygwin includes >> coreutils, so has mkfifo, and that command does something. However, >> the resultant named pipe is known (on the Cygwin mailing list at >> least) to not work correctly. > > Hm. How would you recommend going about writing a script that takes > output from a command, transforms it, and then feeds it back into > that command's input? Are sockets a more reliable way to do this kind > of IPC on Cygwin? > > See reinit_git and try_dump from t9010-svn-fe.sh for context. > > Thanks, > Jonathan t9010 needs PIPE in most test cases as a prerequisite. And if PIPE isn't available, the tests can not be run. Are you suggesting to replace the named pipes with a TCP socket? Disabling PIPE under cygwin seems to be the right thing to do, or do I miss something ? /Torsten -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] test-lib.sh - cygwin does not have usable FIFOs
Mark Levedahl wrote: > Do not use FIFOs on cygwin, they do not work. Cygwin includes > coreutils, so has mkfifo, and that command does something. However, > the resultant named pipe is known (on the Cygwin mailing list at > least) to not work correctly. Hm. How would you recommend going about writing a script that takes output from a command, transforms it, and then feeds it back into that command's input? Are sockets a more reliable way to do this kind of IPC on Cygwin? See reinit_git and try_dump from t9010-svn-fe.sh for context. Thanks, Jonathan -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html