Hi zack, > On Thu, Jul 31, 2014 at 12:50:58AM +1000, Stuart Prescott wrote: >> My proposal is then: >> >> * ensure the internal parser supports whitespace and comments as required >> >> * switch the default parser in iter_paragraphs to the internal parser >> (changing the kwarg to be use_apt_pkg=False) and clearly document the >> situations where an application might safely choose to use TagFile >> (use_apt_pkg=True). > > As a plan, this looks good to me: as a default, I'd favor correctness > over speed anytime. If one is ready to sacrifice correctness for speed, > asking to do so explicitly sounds reasonable. Of course, this change > will warrant a NEWS.Debian entry.
I don't believe it does warrant a NEWS.Debian entry I wasn't planning to include one. python-debian is installed on a huge number of debian systems (with popcon showing it in 76% of responses -- reportbug depends on it so it is effectively priority standard even if that's a priority mismatch!). Of those 76%, most don't know it's even installed and almost none of them care because they're not writing code with deb822.py. We wouldn't include a NEWS item if this were a non-native library changing its internal behaviour in a backwards-compatible way and I'm pretty sure we shouldn't reduce the signal:noise in apt-listchanges. cheers Stuart -- Stuart Prescott http://www.nanonanonano.net/ [email protected] Debian Developer http://www.debian.org/ [email protected] GPG fingerprint 90E2 D2C1 AD14 6A1B 7EBB 891D BBC1 7EBB 1396 F2F7 -- http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-python-debian-maint
