[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1422706 - in /subversion/trunk/subversion: libsvn_subr/opt.c tests/cmdline/svntest/main.py

From: Branko Čibej <brane_at_wandisco.com>
Date: Mon, 17 Dec 2012 11:19:08 +0100

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
Received on 2012-12-17 11:19:48 CET

This is an archived mail posted to the Subversion Dev mailing list.