hi john,

On Sat, Jul 25, 2009 at 07:38:57PM +0200, John Wright wrote:
> > A temporary workaround is to pass use_apt_pkg=False to the
> > iter_paragraphs method.  On large files, you'll probably notice a bit of
> > a performance hit.  I'll see what we can do to preserve this information
> > with apt_pkg.

yes, that seems to make the problem go away.

> I'm actually inclined to turn off using apt_pkg by default.  It's
> definitely faster, typically by a factor between 2 and 2.5, but we keep
> running into weird corner cases with the way apt_pkg parses things.

for me anyway, speed is not a huge concern as the code path occurs during
a behind-the-scenes cronjob.  i guess in an ideal world apt_pkg would
correctly parse the file and it'd be a non-issue, but barring that i'd
throw my vote in for the slower but correct implementation as well... if
only because i don't want to have to support two different versions of
deb822.Sources in my code (production is running on lenny, development
on squeeze/sid).


        sean

-- 

Attachment: signature.asc
Description: Digital signature

Reply via email to