[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r14756 - branches/1.2.x

From: Peter N. Lundblad <peter_at_famlundblad.se>
Date: 2005-05-17 09:13:03 CEST

On Mon, 16 May 2005, Greg Hudson wrote:

> On Mon, 2005-05-16 at 15:10 -0500, lundblad@tigris.org wrote:
> > + Don't initialize libsvn_ra in svnversion:
> > + Justification:
> > + svnversion doesn't use the RA layer, and it only depends on it
> > + indirectly.
> > + Votes:
> > + +1: lundblad
>
> (1) You didn't include any justification in the log message for r14755.
> This justification is the only clue we have as to why you made that
> change.
>
You're right.

> (2) Who cares if svnversion only depends on the RA layer indirectly? We
> don't have any indirect functions for initializing it.
>
So you want svnversion to directly depend on libsvn_ra? I think the
current solution is OK for 1.2. We need to work htis out during 1.3.

I'll fix the commit message.

Regards,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue May 17 09:04:42 2005

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.