Merge lp:~gz/brz/py3_ignores into lp:brz
Status: | Merged |
---|---|
Approved by: | Jelmer Vernooij |
Approved revision: | no longer in the source branch. |
Merged at revision: | 6811 |
Proposed branch: | lp:~gz/brz/py3_ignores |
Merge into: | lp:brz |
Diff against target: |
234 lines (+52/-64) 2 files modified
breezy/ignores.py (+11/-20) breezy/tests/test_ignores.py (+41/-44) |
To merge this branch: | bzr merge lp:~gz/brz/py3_ignores |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jelmer Vernooij | Approve | ||
Review via email: mp+333588@code.launchpad.net |
Commit message
Make test_ignores (mostly) pass on Python 3
Description of the change
This leaves one failing test on Python 3:
=======
FAIL: breezy.
-------
...
Traceback (most recent call last):
File "/home/
self.
File "/home/
pprint.
AssertionError: not equal:
a = 'myentry1\
b = 'myentry1\
This is a real regression in behaviour, but will want to be fixed by dropping reliance on the deprecated 'U' flag to open and 'newlines' attribute and using an alternative scheme to preserve the newline type of existing ignore files.