Merge lp://staging/~mvo/python-apt/mvo into lp://staging/~mvo/python-apt/debian-sid-mirrored

Proposed by Michael Vogt
Status: Merged
Merged at revision: 579
Proposed branch: lp://staging/~mvo/python-apt/mvo
Merge into: lp://staging/~mvo/python-apt/debian-sid-mirrored
Diff against target: 28 lines (+6/-1)
2 files modified
apt/package.py (+4/-1)
debian/changelog (+2/-0)
To merge this branch: bzr merge lp://staging/~mvo/python-apt/mvo
Reviewer Review Type Date Requested Status
Julian Andres Klode (community) Approve
Review via email: mp+99483@code.staging.launchpad.net

Description of the change

This will make the Version.uri method return None if there is no uri for this version instead of a StopIteration exception. This seems conceptually cleaner.

To post a comment you must log in.
Revision history for this message
Julian Andres Klode (juliank) wrote :

That seems fine.

review: Approve
Revision history for this message
Michael Vogt (mvo) wrote :

On Tue, Mar 27, 2012 at 09:30:36AM -0000, Julian Andres Klode wrote:
> Review: Approve
>
> That seems fine.

Great, thanks a bunch.

Cheers,
 Michael

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

to all changes: