Re: [PATCH] '--version --quiet' should display only the version info in svnadmin, svnlook, svnsync, svndumpfilter, svnversion, svnserve

Prabhu Gnana Sundar wrote on Thu, Jan 06, 2011 at 08:02:09 +0530:
> Hi Daniel,
>
> Since I am just now getting my feet wet with the subversion code, I am
> finding opportunities in everything I see... :)
>
> On Wed, 2011-01-05 at 15:11 +0200, Daniel Shahaf wrote:
> > Not to say that this patch is unnecessary (maybe you have a usecase
> > wherein it's required), but if you're looking to cut your teeth, I can
> > suggest a few more challening things to work on. :-)
> >
>
> I would be really happy to get challenging suggestions from you :)
>

Okay. Here are some suggestions. (Most of them have been recorded in the
issue tracker at some point.) They are in various parts of the codebase
and some or more difficult^Wchallenging than others.

* (from IRC yesterday) In SVNListParentPath, don't list repositories
  that aren't readable by the authenticated user.

  (Currently it also lists things which aren't repositories at all.)

* Fix issue #3398 (re dynamic library loading).

* Fix issue #3532 (wc assert with ra_serf).

* Fix issue #3767 (ra_dav error marshalling).

* Fix issue #3641 (svnsync/svnrdump handling of nested replace-with-history).

Some of these are on my list too and I'll try to review patches
for them, for others I make no promises.

>
> Thanks and regards
> Prabhu

Good luck,

Daniel
Received on 2011-01-06 19:05:19 CET