Fix committed in r4788.

Yep, works a treat

Does adding a call to 'logging.shutdown()' at the end of that
testcase make Windows happy?

Nearly there... you hit the same problem I did. We need to close the log file.

ERROR: test_sample_config_w_log_file (repoze.who.tests.test_config.TestConfigMid
dleware)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\code\repoze.who\repoze\who\tests\test_config.py", line 337, in tearDo
wn
    shutil.rmtree(self.tempdir)
  File "c:\python25\lib\shutil.py", line 180, in rmtree
    onerror(os.remove, fullname, sys.exc_info())
  File "c:\python25\lib\shutil.py", line 178, in rmtree
    os.remove(fullname)
WindowsError: [Error 32] The process cannot access the file because it is being
used by another process: 'c:\\docume~1\\paul\\locals~1\\temp\\tmpwtmexf\\who.log
'

----------------------------------------------------------------------
Ran 206 tests in 2.688s

FAILED (errors=1)

I plan to check in an alternate patch which uses 'tempfile.mkdtemp',
with appropriate cleanup of the tempdir.  Can you confirm that it
works for you on Windows?

Ok, here's a patch. test_sample_config_w_log_file in test_config does now leave
the log file existing; can't see a way to fix that.

Just updated to latest code base and the problem is a bit worse. working on a
new patch

One unit test fails on Windows - problem with the test, not the code base.