Stefane Fermigier wrote: >Jean-Marc Orliaguet wrote: > > >>Stefane Fermigier wrote: >> >> >> >>>We had a long-standing unit test error on CPSSkins: >>> >>>Failure in test testMsgExists >>>(Products.CPSSkins.tests.testI18N.TestOneMsg) >>>Traceback (most recent call last): >>> File "./lib/python/Testing/ZopeTestCase/profiler.py", line 98, in >>>__call__ >>> File >>>"/home/fermigier/svn/CPS3/zope/Products/CPSSkins/tests/testI18N.py", line >>>164, in testMsgExists >>> self.fail(msg) >>> File >>>"/home/fermigier/svn/CPS3/3rdparty/Python/../../lib/python2.4/unittest.py >>>", line 301, in fail >>> raise self.failureException, msg >>>AssertionError: Comparing 'cpsskins-it.po' with 'cpsskins.pot' raised an >>>error, >>> exit code of msgcmp is: 256 >>>/home/fermigier/svn/CPS3/zope/Products/CPSSkins/tests/../i18n/cpsskins.pot:1975: >>> >>>this message is used but not defined... >>>/home/fermigier/svn/CPS3/zope/Products/CPSSkins/tests/../i18n/cpsskins-it.po:160 >>> >>>7: ...but this definition is similar >>>msgcmp: found 1 fatal error >>> >>>I think I fixed the mistake in commit >>>http://svn.nuxeo.org/trac/pub/changeset/31300 >>> >>>Unfortunately it only appeared on platforms where msgcmp (Debian package >>>'gettext') was installed, which caused some difficulties on finding it. >>> >>>Unfortunately (II) it was burried in the flurry of "2006-01-04 12:54:52 >>>ERROR Zope.ZCatalog uncatalogObject unsuccessfully attempted to >>>uncatalog an object with a uid of xxx" that the unit tests spit (as I >>>asked before, is there any way to either suppress the problem or silence >>>the message since this causes serious problems to go unnotified?). >>> >>>Another related question: I notice that the CPSSkins package has a "make >>>i18n" option that, IIUC, would have prevented this problem to appear in >>>the first place. But how do we make sure that this is always done ? >>> >>> >>> >>> >>> >>Hi! >> >>running: >> >>$ make i18n >> >>in the CPSSkins directory solves the issue, it makes sure that all msgid >>are translated (it will add empty message strings). It should be run >>when translations are updated for the tests to pass. >> >>cf. >>http://svn.nuxeo.org/trac/pub/file/CPSSkins/trunk/doc/adding-missing-translations.txt >> >> > >1. This doc doesn't point to the right repositoy, but I will change it. > > > fixed in svn >2. With a fresh SVN checkout, I get an error with 'make i18n', the same >I got with i18ndude version 2.0. I suppose you have been using version >1.0 or earlier. Is it possible to update to this version ? > > > fixed too.. (remove the -s option) >3. I'm still wondering why we're not using this approach for every CPS >package with i18n needs. > > S. > > > I think that the CPSUtil i18n utility does that already but it can't be run on the CPSSkins i18n files since there are more than 1 pot file. /JM
Hosting: Nuxeo: Zope service provider