Hi again, Dmitry Ivankov writes: > svn-fe needs to read fast-import's responses to "ls" and "cat-blob". > These come through a file descriptor number 3. > > Sometimes it is easier to setup variable fd than a fixed one. It is > the case with pipe() call and even more fd=3 can be already taken. > On Windows file descriptors are not by default inherited by a child > process, nor there is an option to setup descriptors other than > standard stdin, stdout, stderr at a process creation time. > > Add an option for this file descriptor number in vcs-svn/ and svn-fe, > add a simple test for it. > > To be used like following: > $ svn-fe --read-blob-fd=7 ... 7<somewhere Except for the fact that you're changing the API to accommodate an extra option as I already pointed out earlier, I like this patch. This was one of the things that I'd wanted to do myself, but never actually got around to doing. Thanks for working on this. -- Ram -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html