[MERGE] dpush / foreign vcs testing

[Jelmer Vernooij]

Aaron Bentley wrote:
> Russel Winder wrote:
> > I want to highlight the fact that dpush has gone
> > from bzr-svn but is critical to certain important workflows when using
> > Bazaar with Subversion.
>
> That's Jelmer's responsibility, not the bzr team's.  It's certainly easy
> to provide a smoother transition than this, by only registering dpush in
> the plugin if core doesn't already provide it.
>
> I'm sorry you're caught in the middle, but this is Jelmer's choice, so
> it doesn't make me feel any additional urgency.
Yeah, this was my choice and I'm not sure it was a good one. The problem
is that I didn't want to end up with three different copies of the dpush
command that I had to keep up to date.

An obvious choice would've been to move dpush to a separate plugin for
the time being, but that seemed overkill for something that was waiting
in the merge queue for bzr.dev. So, given the fact that there is only a
small number of users of dpush and the fact that bzr-svn's trunk
occasionally breaks anyway because of API changes in bzr.dev (it is a
development branch, after all), I opted to remove dpush from bzr-svn for
the time being and wait for dpush to either land in bzr.dev or until I
had to move it to a separate plugin.

Anyway, John has just reviewed the dpush patch (*very* much appreciated)
so this should soon be a problem of the past.

Cheers,

Jelmer