Bazaar Fast Import

Merge lp://qastaging/~andersk/bzr-fastimport/git-darcs into lp://qastaging/bzr-fastimport

git-darcs
Merge into trunk

Proposed by Anders Kaseorg on 2010-03-27

Status:	Merged
Approved by:	Jelmer Vernooij on 2010-11-09
Approved revision:	282
Merged at revision:	283
Proposed branch:	lp://qastaging/~andersk/bzr-fastimport/git-darcs
Merge into:	lp://qastaging/bzr-fastimport
Diff against target:	284 lines (+46/-55) 2 files modified exporters/darcs/darcs-fast-export (+10/-19) exporters/darcs/git-darcs (+36/-36)
To merge this branch:	bzr merge lp://qastaging/~andersk/bzr-fastimport/git-darcs
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Miklos Vajna	code	2010-10-09	Pending
Ian Clatworthy		2010-03-27	Pending
Review via email: mp+22279@code.qastaging.launchpad.net

Description of the change

Some fixes for git-darcs, including Daniel Herring’s patch from bug 538933.

Revision history for this message

Jelmer Vernooij (jelmer) wrote on 2010-10-09:

The git-darcs changes are fairly simple and seem ok to me.

Miklos, you are a lot more familiar with the darcs exporter code. Any chance you can do a review of that code?

Revision history for this message

Miklos Vajna (vmiklos) wrote on 2010-10-12:

On Sat, Oct 09, 2010 at 01:31:38PM -0000, Jelmer Vernooij <email address hidden> wrote:
> Miklos, you are a lot more familiar with the darcs exporter code. Any
> chance you can do a review of that code?

Sure, I'll try to review it in the near future. :)

Revision history for this message

Miklos Vajna (vmiklos) wrote on 2010-11-08:

On Wed, Oct 13, 2010 at 01:49:57AM +0200, Miklos Vajna <email address hidden> wrote:
> Sure, I'll try to review it in the near future. :)

Sorry for being slow, looks good.

Jelmer, please merge it.

Thanks.

Revision history for this message

Jelmer Vernooij (jelmer) wrote on 2010-11-09:

Thanks for the review.