You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openoffice.apache.org by Pavel Janík <Pa...@Janik.cz> on 2013/03/03 13:03:38 UTC

Re: svn commit: r1451887 - /openoffice/trunk/main/i18npool/source/localedata/data/es_PE.xml

Hi,

this breaks the build:

Warning: Time100SecSeparator is different from DecimalSeparator, this may be correct or not. Intended?
Warning: Don't forget to adapt corresponding FormatCode elements when changing separators.
Error: Ordering of ThousandSeparator and DecimalSeparator not correct in formatindex="4".
Warning: formatindex="4","44","45" are the only FormatCode elements checked for separator usage, there may be others that have errors.
Error: in data for es_PE: 1
parsing document es_PE finished
dmake:  Error code 1, while making '../../../unxmacxi.pro/misc/localedata_es_PE.cxx'
ERROR: error 65280 occurred while making /Users/pavel/BUILD/BuildDir/ooo_trunk_src/i18npool/source/localedata/data


On Mar 2, 2013, at 2:40 PM, jza@apache.org wrote:

> Author: jza
> Date: Sat Mar  2 13:40:33 2013
> New Revision: 1451887
> 
> URL: http://svn.apache.org/r1451887
> Log:
> new xml
> 
> Modified:
>    openoffice/trunk/main/i18npool/source/localedata/data/es_PE.xml
> 
> Modified: openoffice/trunk/main/i18npool/source/localedata/data/es_PE.xml
> URL: http://svn.apache.org/viewvc/openoffice/trunk/main/i18npool/source/localedata/data/es_PE.xml?rev=1451887&r1=1451886&r2=1451887&view=diff
> ==============================================================================
> --- openoffice/trunk/main/i18npool/source/localedata/data/es_PE.xml (original)
> +++ openoffice/trunk/main/i18npool/source/localedata/data/es_PE.xml Sat Mar  2 13:40:33 2013
> @@ -37,8 +37,8 @@
> <LC_CTYPE unoid="generic">
> <Separators>
> <DateSeparator>/</DateSeparator>
> -<ThousandSeparator>.</ThousandSeparator>
> -<DecimalSeparator>,</DecimalSeparator>
> +<ThousandSeparator>,</ThousandSeparator>
> +<DecimalSeparator>.</DecimalSeparator>
> <TimeSeparator>:</TimeSeparator>
> <Time100SecSeparator>,</Time100SecSeparator>
> <ListSeparator>;</ListSeparator>
> 
> 

-- 
Pavel Janík




Re: svn commit: r1451887 - /openoffice/trunk/main/i18npool/source/localedata/data/es_PE.xml

Posted by Pavel Janík <Pa...@Janik.cz>.
>>>> +<ThousandSeparator>,</ThousandSeparator>
>>>> +<DecimalSeparator>.</DecimalSeparator>
>> 
>> Is there a bug number for this commit?
> 
> Found bug 121825
> Should be fixed with revision 1452033

and verified it builds now.

Thanks!
-- 
Pavel Janík




Re: svn commit: r1451887 - /openoffice/trunk/main/i18npool/source/localedata/data/es_PE.xml

Posted by Ariel Constenla-Haile <ar...@apache.org>.
On Sun, Mar 03, 2013 at 09:24:28AM -0300, Ariel Constenla-Haile wrote:
> Hi,
> 
> On Sun, Mar 03, 2013 at 01:03:38PM +0100, Pavel Janík wrote:
> > Hi,
> > 
> > this breaks the build:
> > 
> > Warning: Time100SecSeparator is different from DecimalSeparator, this may be correct or not. Intended?
> > Warning: Don't forget to adapt corresponding FormatCode elements when changing separators.
> > Error: Ordering of ThousandSeparator and DecimalSeparator not correct in formatindex="4".
> 
> <FormatElement msgid="FixedFormatskey5" default="false" type="medium" usage="FIXED_NUMBER"  formatindex="4">
> <FormatCode>#.##0,00</FormatCode>
> <DefaultName></DefaultName>
> </FormatElement>
> 
> Indeed: the format code #.##0,00 has point as ThousandSeparator and
> comma as DecimalSeparator, while
> 
> > > +<ThousandSeparator>,</ThousandSeparator>
> > > +<DecimalSeparator>.</DecimalSeparator>
> 
> Is there a bug number for this commit?

Found bug 121825
Should be fixed with revision 1452033


Regards
-- 
Ariel Constenla-Haile
La Plata, Argentina

Re: svn commit: r1451887 - /openoffice/trunk/main/i18npool/source/localedata/data/es_PE.xml

Posted by Ariel Constenla-Haile <ar...@apache.org>.
Hi,

On Sun, Mar 03, 2013 at 01:03:38PM +0100, Pavel Janík wrote:
> Hi,
> 
> this breaks the build:
> 
> Warning: Time100SecSeparator is different from DecimalSeparator, this may be correct or not. Intended?
> Warning: Don't forget to adapt corresponding FormatCode elements when changing separators.
> Error: Ordering of ThousandSeparator and DecimalSeparator not correct in formatindex="4".

<FormatElement msgid="FixedFormatskey5" default="false" type="medium" usage="FIXED_NUMBER"  formatindex="4">
<FormatCode>#.##0,00</FormatCode>
<DefaultName></DefaultName>
</FormatElement>

Indeed: the format code #.##0,00 has point as ThousandSeparator and
comma as DecimalSeparator, while

> > +<ThousandSeparator>,</ThousandSeparator>
> > +<DecimalSeparator>.</DecimalSeparator>

Is there a bug number for this commit?


Regards
-- 
Ariel Constenla-Haile
La Plata, Argentina