Merge lp://staging/~vila/bzr/660935-invalid-entry-name into lp://staging/bzr/2.2

Proposed by Vincent Ladeuil
Status: Merged
Approved by: John A Meinel
Approved revision: no longer in the source branch.
Merged at revision: 5118
Proposed branch: lp://staging/~vila/bzr/660935-invalid-entry-name
Merge into: lp://staging/bzr/2.2
Diff against target: 173 lines (+95/-4)
3 files modified
NEWS (+3/-0)
bzrlib/conflicts.py (+3/-3)
bzrlib/tests/test_conflicts.py (+89/-1)
To merge this branch: bzr merge lp://staging/~vila/bzr/660935-invalid-entry-name
Reviewer Review Type Date Requested Status
John A Meinel Approve
Vincent Ladeuil Approve
Review via email: mp+46348@code.staging.launchpad.net

Commit message

Correctly resolve --take-this/--take-other for path/content conflicts in subdirs.

Description of the change

Trivial fix for resolving conflicts in subdirs.

The shell-script test will go away when I merged in 2.3 and trunk but I thought it clearly explain the bug and may help during the SRU process.

To post a comment you must log in.
Revision history for this message
Vincent Ladeuil (vila) wrote :

Vocally approved by jam :)

review: Approve
Revision history for this message
Vincent Ladeuil (vila) wrote :

sent to pqm by email

Revision history for this message
John A Meinel (jameinel) wrote :

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

On 1/14/2011 3:43 PM, Vincent Ladeuil wrote:
> Vincent Ladeuil has proposed merging lp:~vila/bzr/660935-invalid-entry-name into lp:bzr/2.2.
>
> Requested reviews:
> bzr-core (bzr-core)
> Related bugs:
> #660935 bzrlib.errors.InvalidEntryName when resolving path conflict
> https://bugs.launchpad.net/bugs/660935
>
> For more details, see:
> https://code.launchpad.net/~vila/bzr/660935-invalid-entry-name/+merge/46348
>
> Trivial fix for resolving conflicts in subdirs.
>
> The shell-script test will go away when I merged in 2.3 and trunk but I thought it clearly explain the bug and may help during the SRU process.
 merge: approve

Though I'm wondering why I didn't see you vote this up already. I would
like to see the script test case removed at some point.

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

iEYEARECAAYFAk0wz44ACgkQJdeBCYSNAAOkngCfWYPc1CcgpOV+zkcazHTEnvTH
wJ0AoJLHkJnGPCSQcrMH6oG7qAAHmhSR
=o7OI
-----END PGP SIGNATURE-----

review: Approve

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