[Unicode]   Common Locale Data Repository : Bug Tracking Home | Site Map | Search
 
Modify

CLDR Ticket #11025(closed: fixed)

Opened 8 months ago

Last modified 8 days ago

SupplementalMapper parses dates using the default locale

Reported by: roubert Owned by: mark
Component: infrastructure Data Locale:
Phase: dsub Review: roubert
Weeks: Data Xpath:
Xref:

Description (last modified by roubert) (diff)

SupplementalMapper.getMilliSeconds() is supposed to parse dates in a standardized format.

But it currently calls the default constructor of SimpleDateFormat() which uses the default locale (in my own case set by the value of the LC_CTYPE environment variable) to determine the parsing behaviour.

Because of this, building ICU data from CLDR fails in every environment where the default locale doesn't parse like the root locale.

[cldr-build] java.lang.IllegalArgumentException: Could not parse date: -2636-01-01
[cldr-build] 	at org.unicode.cldr.icu.SupplementalMapper.getSeconds(SupplementalMapper.java:369)

The attached patch solves this problem by explicitly passing ULocale.ROOT to the SimpleDateFormat constructor.

GoogleIssue:76112294

Attachments

cldr-ticket-11025-patch-1.diff (1.2 KB) - added by roubert 8 months ago.

Change History

Changed 8 months ago by roubert

comment:1 Changed 8 months ago by roubert

  • Description modified (diff)

comment:2 Changed 7 months ago by mark

  • Keywords google added

comment:3 Changed 5 months ago by mark

  • Owner changed from anybody to mark
  • Phase changed from dsub to dvet
  • Priority changed from assess to major
  • Status changed from new to accepted
  • Milestone changed from UNSCH to 34

comment:4 Changed 3 months ago by mark

  • Phase changed from dvet to dsub
  • Milestone changed from 34 to 35

comment:5 Changed 3 weeks ago by mark

  • Component changed from util to infrastructure

comment:6 Changed 8 days ago by mark

  • Status changed from accepted to reviewing
  • Review set to roubert

comment:7 Changed 8 days ago by roubert

  • Status changed from reviewing to closed
  • Resolution set to fixed
View

Add a comment

Modify Ticket

Action
as closed
Next status will be 'new'
Next status will be 'closed'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.