Code review comment for lp:~brian-murray/apport/retrace-package-versions

Revision history for this message
Martin Pitt (pitti) wrote :

Also, in make_sandbox()'es second hunk it only checks report['Package'] but not 'Dependencies'? Should mostly be harmless though. But the inner if is missing an else path, otherwise we won't append the package at all.

Looks good to me otherwise, thanks!

« Back to merge proposal