Bazaar

Merge lp://qastaging/~jameinel/bzr/2.4-testament-binary-1010339 into lp://qastaging/bzr/2.4

  1. 2.4-testament-binary-1010339
  2. Merge into 2.4
Proposed by John A Meinel
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: no longer in the source branch.
Merged at revision: 6071
Proposed branch: lp://qastaging/~jameinel/bzr/2.4-testament-binary-1010339
Merge into: lp://qastaging/bzr/2.4
Diff against target: 91 lines (+24/-2)
4 files modified
bzrlib/btree_index.py (+6/-0)
bzrlib/builtins.py (+3/-2)
bzrlib/tests/blackbox/test_testament.py (+11/-0)
doc/en/release-notes/bzr-2.4.txt (+4/-0)
To merge this branch: bzr merge lp://qastaging/~jameinel/bzr/2.4-testament-binary-1010339
Reviewer Review Type Date Requested Status
Vincent Ladeuil Needs Information
Alexander Belchenko Approve
Review via email: mp+109301@code.qastaging.launchpad.net

Commit message

Fix bug #1010339, use encoding_type='exact' for bzr testament

Description of the change

A bugfix for bug #1010339. Basically, we weren't setting O_BINARY for sys.stdout for 'bzr testament'.

The test here isn't quite good enough, because it doesn't spawn a subprocess to actually test the output. However, that is only useful on Windows.

The difference is that if you do change sys.stdout to self.outf, then the test will fail because of double encoding. So it does push us towards using encoding_type = 'exact'.

I'm targetting 2.4, because this seems a simple bugfix. We don't have to backport it, because it certainly isn't a regression, but the number of touched lines is quite small.

To post a comment you must log in.
Revision history for this message
Alexander Belchenko (bialix) wrote :

John A Meinel пишет:
> John A Meinel has proposed merging lp:~jameinel/bzr/2.4-testament-binary-1010339 into lp:bzr/2.4.
>
> A bugfix for bug #1010339. Basically, we weren't setting O_BINARY for sys.stdout for 'bzr testament'.

The first hunk (changes in btree_index.py re new_leaf) looks suspicious
because it does not fit your description. Maybe it's also important fix,
but seems unrelated to stdout.

--
All the dude wanted was his rug back

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

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

On 6/8/2012 9:21 AM, Alexander Belchenko wrote:
> John A Meinel пишет:
>> John A Meinel has proposed merging
>> lp:~jameinel/bzr/2.4-testament-binary-1010339 into lp:bzr/2.4.
>>
>> A bugfix for bug #1010339. Basically, we weren't setting O_BINARY
>> for sys.stdout for 'bzr testament'.
>
> The first hunk (changes in btree_index.py re new_leaf) looks
> suspicious because it does not fit your description. Maybe it's
> also important fix, but seems unrelated to stdout.
>

It is. '...' is invalid syntax, and causes problems on python-2.7. It
might just be a warning, but we need to fix it.

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

iEYEARECAAYFAk/RqkYACgkQJdeBCYSNAAPnxwCfS6xj1HDP92RlC0ib/at3X1rc
OTwAoM2D+0xzR9o0d1S+VHXjk9jVPWwk
=NRUF
-----END PGP SIGNATURE-----

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

What the rationale for submitting against 2.4 when the windows stable series is 2.5 ? We won't ever release a 2.4.3 for windows.

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

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

On 6/8/2012 9:40 AM, Vincent Ladeuil wrote:
> Review: Needs Information
>
> What the rationale for submitting against 2.4 when the windows
> stable series is 2.5 ? We won't ever release a 2.4.3 for windows.

Its been a bug for a while, and so I picked something older. No
particular reason.

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

iEYEARECAAYFAk/RvNIACgkQJdeBCYSNAAPINwCgsn30zC4+Rf3S66qyVMuXRpW0
srAAnjYifIrnu94ewXvgxb4/8KBxT9UZ
=TwV5
-----END PGP SIGNATURE-----

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