Merge lp://qastaging/~mvo/ubuntu-release-upgrader/partial-upgrade-cleanup into lp://qastaging/ubuntu-release-upgrader

Proposed by Michael Vogt
Status: Merged
Merge reported by: Iain Lane
Merged at revision: not available
Proposed branch: lp://qastaging/~mvo/ubuntu-release-upgrader/partial-upgrade-cleanup
Merge into: lp://qastaging/ubuntu-release-upgrader
Diff against target: 146 lines (+41/-20)
2 files modified
DistUpgrade/DistUpgradeViewText.py (+12/-4)
do-partial-upgrade (+29/-16)
To merge this branch: bzr merge lp://qastaging/~mvo/ubuntu-release-upgrader/partial-upgrade-cleanup
Reviewer Review Type Date Requested Status
Brian Murray Approve
Ubuntu Core Development Team Pending
Review via email: mp+214685@code.qastaging.launchpad.net

Description of the change

This branch fixes:
- a crash when the DistUpgradeViewText is used with py3
- fixes a bug with the logging setup (LP: #1302380) (thanks for the fix to Brian Murray and https://code.launchpad.net/~brian-murray/ubuntu-release-upgrader/bug-1302380/+merge/214349)
- fixes a crash when do-partial-upgrade is used without a DISPLAY

To post a comment you must log in.
Revision history for this message
Brian Murray (brian-murray) wrote :

It looks to me like the readline change with fix bug 1243619.

Revision history for this message
Brian Murray (brian-murray) wrote :

This looks good to me, but I reference bug 1243619 if it is in fact fixed by the readline changes.

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