Ævar Arnfjörð Bjarmason wrote: > On Mon, Jul 26, 2010 at 22:32, Greg Brockman <gdb@xxxxxxx> wrote: >> Just sending a reminder about this patch series--I haven't seen any >> comments on it yet, so I assume it's gotten lost in the flurry of >> other list activity. > > It would probably help if you re-send the entire thing again. Wait wait, it’s only been about five days! I mean, you are free to re-send, but it is probably better to send a link to the gmane archive, like this: http://thread.gmane.org/gmane.comp.version-control.git/151398 so people can catch up with the earlier discussion. In this case, I am nervous about the impact for existing installations with git-shell deployed. If a person can smuggle in an unpleasant git-shell-commands directory somehow, the effect would not be good. Maybe there should be a way to disable this feature systemwide for the paranoid (or maybe not; I’m only vaguely worried). Patch 1 still uses execv(), which is not available on Windows. Have you tried out these patches "in the wild"? If so, that would be interesting to hear about. Jonathan -- 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