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

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

I committed the c → cache renaming in http://bazaar.launchpad.net/~apport-hackers/apport/trunk/revision/2790 as it is quite a lot of noise and unrelated to the actual change here.

I'm not happy about the test case, as it now assumes fixed version numbers in precise-updates, which may change in the future. I think it's better (as suggested above) to pin the version down to the precise-final one and confirm that those get installed instead of the latest precise-updates versions. I'll work on that.

Thanks!

« Back to merge proposal