Code review comment for lp://staging/~mvo/software-center/hopefully-fix-lp1008229

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

On Wed, Jul 11, 2012 at 01:19:19AM -0000, Gary Lasker wrote:
> Review: Approve

Thanks for the review.

> Yep, looking at the code it seems that this would only happen on an
> error opening the deb file, so this fix should take care of it. I
> wonder why so many people are hitting this though.
[..]

Yes, its a bit puzzling as I did not find a way to reproduce it yet.

Cheers,
 MIchael

« Back to merge proposal