Re: [PATCH/RFC v2 0/9] Subversion dump parsing library

From: Ramkumar Ramachandra <artagnon@gmail.com>
Date: Jun 24, 2010, 13:06 UTC
Message-ID: <AANLkTilGNaF05EW2IFUpjOx-lP8sDoCkSrsa1-vMZo6B@mail.gmail.com>
In-Reply-To: <20100624105004.GA12336@burratino>

(+CC: Junio)

Hi Jonathan,

Jonathan Nieder wrote:

> Ram last sent this series a couple of weeks ago[1], and it was
> merged to pu then as rr/svn-export.  Here’s another iteration
> of the same for discussion, now including David Barr’s program
> that demonstrates the functionality.

There's a lot of work happening in the exporter and the series will
probably need to be re-rolled again: I recommend the following.
1. Review this series thoroughly, but don't actually merge it because
it's going to be re-rolled soon.
2. Split the series in two: The infrastructure part in vcs-svn/ will
be re-rolled later. Another part for contrib/ should be made into a
separate series and merged.
3. As soon as the client is complete, I'll roll a series that puts it
in vcs-svn/ (infrastructure again).
4. Finally, I'll roll out a series for the remote helper itself that
puts it in $GIT_ROOT.

To summarize, there should be four series:
1. David's exporter in vcs-svn/ (yet to be re-rolled): Either Jonathan
or I will handle this.
2. David's independent svn-fe program in contrib/: Jonathan will handle this.
3. The RA client to generate a full-text dumpfile on-the-fly: This
isn't done yet; as soon as it's finished, I'll roll a series.
4. The remote helper: This done, but it doesn't make sense to send
this in before everything else has been merged.

-- Ram