apturl

Merge lp://qastaging/~laney/apturl/1726327 into lp://qastaging/apturl

  1. 1726327
  2. Merge into ubuntu
Proposed by Iain Lane
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 150
Merged at revision: 150
Proposed branch: lp://qastaging/~laney/apturl/1726327
Merge into: lp://qastaging/apturl
Diff against target: 30 lines (+9/-2)
2 files modified
AptUrl/Parser.py (+2/-2)
debian/changelog (+7/-0)
To merge this branch: bzr merge lp://qastaging/~laney/apturl/1726327
Reviewer Review Type Date Requested Status
Sebastien Bacher Approve
Review via email: mp+332625@code.qastaging.launchpad.net
To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote :

see the bug, I think it's because a trailing slash is being added

Revision history for this message
Sebastien Bacher (seb128) wrote :

that seems to address bug #1453576 as well

Revision history for this message
Sebastien Bacher (seb128) wrote :

change looks good to me

review: Approve
Revision history for this message
Will Cooke (willcooke) wrote :

Tested with xdg-open on Arful, and it has the desired effect.

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