Re: [PATCHv3] Updated patch series for providing mechanism to list available repositories

From: Jonathan Nieder <jrnieder@gmail.com>
Date: Jul 26, 2010, 23:28 UTC
Message-ID: <20100726232855.GA3157@burratino>
In-Reply-To: <AANLkTilSqePFPkteFd7DBgmdhqJHfUDuW_qhkbWVVb3Y@mail.gmail.com>

Ævar Arnfjörð Bjarmason wrote:

> On Mon, Jul 26, 2010 at 22:32, Greg Brockman <gdb@mit.edu> wrote:

Show 5 quoted lines

>> 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