Merge lp://qastaging/~jelmer/bzr-cvs/controldir into lp://qastaging/bzr-cvs

Proposed by Jelmer Vernooij
Status: Merged
Approved by: John A Meinel
Approved revision: no longer in the source branch.
Merged at revision: 4
Proposed branch: lp://qastaging/~jelmer/bzr-cvs/controldir
Merge into: lp://qastaging/bzr-cvs
Diff against target: 70 lines (+37/-10)
1 file modified
__init__.py (+37/-10)
To merge this branch: bzr merge lp://qastaging/~jelmer/bzr-cvs/controldir
Reviewer Review Type Date Requested Status
John A Meinel Approve
Review via email: mp+52337@code.qastaging.launchpad.net

Description of the change

Support the new ControlDir/Prober API introduced in bzrlib 2.3

To post a comment you must log in.
Revision history for this message
John A Meinel (jameinel) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 3/6/2011 8:25 PM, Jelmer Vernooij wrote:
> Jelmer Vernooij has proposed merging lp:~jelmer/bzr-cvs/controldir into lp:bzr-cvs.
>
> Requested reviews:
> Bazaar Developers (bzr)
>
> For more details, see:
> https://code.launchpad.net/~jelmer/bzr-cvs/controldir/+merge/52337
>
> Support the new ControlDir/Prober API introduced in bzrlib 2.3

TBH, I'm really not familiar with 'bzr-cvs'. Until today, I didn't know
it existed.

As such, I'm happy for you to land this, but I don't really know any
background.

The change looks reasonable to me. What about backwards compatibility,
though?

 merge: approve

John
=:->

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk105EAACgkQJdeBCYSNAAMUlgCgxb47bYrBNdntbIE0biUqpyOT
OD4An3HkyxJvScwjMistAYFdAekRDoUk
=qmg6
-----END PGP SIGNATURE-----

review: Approve
4. By Jelmer Vernooij

Merge support for the new controldir API.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: