Re: In-Repo-Authz

Ben Reser wrote:

> Julian Foad wrote:
>> Just a thought.  Some of these changes are significant (in both
>> lines touched and functionality) on their own.  One that stands
>> out is "svnserve --config-file no longer caches the password and
>> authz db ...".  Would it be a good idea (and not too much work)
>> to merge that first, as a separate change?
>
> I think it would be moderately painful to do that first since the code
> I changed to effect that was the same code I'd already touched to add
> in-repo-authz for svnserve before I realized I should just remove the
> whole caching of password and authz db on startup entirely.

OK, probably not worth it then.

> What value do you think merging the --config-file change first would have

Only the standard general benefits of committing separate changes separately, such as being able to revert one without another, or backport one without another, or review one at a time.

- Julian
Received on 2012-12-19 17:31:33 CET