[Piuparts-devel] Bug#668863: Make piuparts-slave.py default to sid

Andreas Beckmann debian at abeckmann.de
Sun Apr 15 14:35:32 UTC 2012


On 2012-04-15 16:12, Holger Levsen wrote:
>> Without the patch, Section.run() terminates with an unhandled
>> exception, due to a reference the unassigned variable 'distro' (line
>> 389), if the 'distro' configuration parameter is not defined.
> 
> Did you test this with an upgrade configuration, like testing2sid or 
> squeeze2wheezy? I seem to recall that setting "distro" for these tests will 
> cause unwanted consequences.

setting both distro and upgrade-test-distros is not problematic (at
least in my heavily patched instance), but I think a section without
both distro and upgrade-test-distros is not well-formed ... defaulting
to something unrelated may cause more problems for upgrade tests.
I'll fix the slave to print an error and skip the bad section.


Andreas





More information about the Piuparts-devel mailing list