On 17.12.2012 09:47, Bert Huijben wrote: > I think we should suppress this warning on Windows when the CryptoAPI > encryption is enabled (read: +- always) to avoid unneeded user questions.
I rather think we should disable plaintext password storage by default, I don't care about Windows, and let people enable it explicitly. > And I’m not sure what to think of the color escape strings... > Do these work on any not Windows os with any terminal settings? These aren't used at all on Windows. They work everywhere on Unix. > (We currently only use these in the test suite and the result is plainly > visisble on the buildbot log) That's funny. if not sys.stdout.isatty() or sys.platform == 'win32': TextColors.disable() They're unconditionally disabled on Windows, so if stdout.isatty() returns True, I don't know /what/ the buildbot does. -- Brane -- Branko Čibej Director of Subversion | WANdisco | www.wandisco.com
