2011-10-31 17:02:52 Tres Seaver napisaĆ(a): > Hmm, there is code in the test which tries to ensure that the locale > for the 'svn' process is set to 'C':: > > $ export ZSVN=svn+ssh://svn.zope.org/repos/main > $ svn log -r85956 $ZSVN/zope.app.applicationcontrol/trunk > r85956 | lgs | 2008-05-01 03:37:07 -0400 (Thu, 01 May 2008) | 1 line > > Add a test and a fix for bug 177733, that was causing ZopeVersion to > report an unknown version when running on a machine with a locale > different than english > $ svn diff -c 85956 $ZSVN/zope.app.applicationcontrol/trunk > ...(elide add of now-failing test) > --- src/zope/app/applicationcontrol/zopeversion.py...85955) > +++ src/zope/app/applicationcontrol/zopeversion.py...85956) > @@ -63,7 +63,7 @@ > def _getSVNInfoOutput(self): > try: > proc = subprocess.Popen('svn info "%s"' % self.path, > - shell=True, stdout=subprocess.PIPE) > + shell=True, stdout=subprocess.PIPE, env={'LANG': 'C'}) > except OSError: > pass > else: > > I guess it is not clever enough. Nevertheless, that test was not > failing on any buildslave before yesterday's report.
Variables are checked in the following order: LC_ALL, LC_MESSAGES, LANG. I suggest to override LC_ALL in test suite. -- Arfrever Frehtes Taifersar Arahesis
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Zope-Dev maillist - Zope-Dev@zope.org https://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - https://mail.zope.org/mailman/listinfo/zope-announce https://mail.zope.org/mailman/listinfo/zope )