Hi,
On Mon, 4 Feb 2008, Nicolas Pitre wrote:
> On Mon, 4 Feb 2008, Johannes Schindelin wrote:
>
> > On Mon, 4 Feb 2008, David Tweed wrote:
> >
> > > + if(c>0 && c<4096 && stat(name, &status) == 0
> > > + && status.st_mtime < now - 600){
> >
> > Please have spaces after the "if" and before the "{" (just imitate the
> > style of the rest of the file).
> >
> > Also, 10 minutes grace period for any ongoing fetch or repack seems a
> > bit arbitrary. Maybe default to 10 minutes, and introduce
> > prune.packGracePeriod?
> >
> > (Which reminds me that it might be useful to add a
> > prune.looseObjectsGracePeriod to avoid having to type --expire= all
> > the time?)
>
> Please use the same parameter for both. There is no need to have
> separate settings.
Right.
Ciao,
Dscho
-
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
