[issue12504] packaging: fix uninstall and stop skipping its tests

[Éric Araujo]

Éric Araujo <merwok at netwok.org> added the comment:

Thanks for that!  I agree that it’s a fake optimization to work with generators instead of lists when the list is small or when we depend on other resources like file handles.  There are a few methods we could change in the database module.

The patch looks good, except that I wouldn’t move code to a nested function, but just change the “yield spam” line to “results.append(spam)”.

In our message, is the test log showing failure something you got before making changes or after?  IOW, does the patch fixes the bug?

----------
assignee: tarek -> eric.araujo
stage:  -> patch review
title: Uninstall has disabled windows tests -> packaging: fix uninstall and stop skipping its tests

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue12504>
_______________________________________