new lines in the changes file Binary: field triggers a parsing error

[Sebastien Bacher]

Trying to sync http://packages.qa.debian.org/m/mono/news/20090907T224401Z.html failed on "2009-09-23 14:20:52 ERROR Exception while processing upload /home/lp_queue/sync-queue/incoming/seb128-20090923-152046 (OOPS-1362U2)", the issue is due to the new line chars in the .changes Binary list, dropping those made the flush-sync works as expected

[Celso Providelo (cprov)]

Related with bug #436182

[Michael Nelson (michael.nelson)]

This should be fixed in devel r9708 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/devel/revision/9708>.

I can't find the above oops to verify whether this bug was caused by the corrupt data in our database producing invalid index files (in which case the work-around is already CP'd in production) or whether it's the result of the initial parsing error (in which case the fix is in r9708 which will be rolled out with LP 3.1.10).

I've chatted with Sebastien and he'll let us know if it re-occurs :)

[Julian Edwards (julian-edwards)]

This happened again: https://pastebin.canonical.com/26803/
With this package: http://packages.qa.debian.org/libx/libxcb/news/20091204T191919Z.html

[Jelmer Vernooij (jelmer)]

This is not so much due to the fact that ChangesFile doesn't accept newlines but more because gina generates/generated invalid changes files.

[Jelmer Vernooij (jelmer)]

This was actually fixed by my changes to use python-debian for gina earlier this year. I'll add some tests to demonstrate this.

[Launchpad QA Bot (lpqabot)]

Fixed in stable r12149 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/12149>.