Hi Mark, I can not reproduce it on my winxp and ubuntu. I believe it caused by the default timezone. would you pls take a look at the current timezone when it fails and whether it passes on RI? thanks a lot!
On 11/7/07, Mark Hindess <[EMAIL PROTECTED]> wrote: > > I'm seeing a failure on Linux where the first test in: > > tests.api.java.util.DateTest.test_toString > > is failing because the result is: > > Thu Jan 01 01:00:00 GMT+01:00 1970 > > rather than: > > Thu Jan 01 00:00:00 GMT 1970 > > Am I the only one seeing this? The test passes on the RI. > > -Mark. > > On 6 November 2007 at 22:36, "Tony Wu" <[EMAIL PROTECTED]> wrote: > > I've committed the patch at r592434. > > the excluded classes are, > > > > tests.api.java.util.CurrencyTest > > tests.api.java.util.FormatterTest > > tests.api.java.util.GregorianCalendarTest > > tests.api.java.util.LocaleTest > > tests.api.java.util.ScannerTest > > > > org.apache.harmony.text.tests.java.text.DecimalFormatSymbolsTest > > org.apache.harmony.text.tests.java.text.NumberFormatTest > > org.apache.harmony.text.tests.java.text.SimpleDateFormatTest > > > > pls kindly let me know if you find any problem, thanks. > > > > On 11/6/07, Tony Wu <[EMAIL PROTECTED]> wrote: > > > On 11/6/07, Tim Ellison <[EMAIL PROTECTED]> wrote: > > > > Tony Wu wrote: > > > > > I've raised a JIRA[1] for migrating the dependencies of locale related > > > > > data to icu4j and remove the data in > > > > > luni/src/main/java/org/apache/harmony/luni/internal/locale/. Currently > > > > > I have delegated all the locale related classes to corresponding > > > > > classes in icu4j and successfully removed the dependencies of harmony > > > > > resource bundles. > > > > > > > > Cool -- this is good because it removes the code duplication we have at > > > > present, it shifts the maintenance of the locale data into the right > > > > place (IMHO) that being the ICU project, and gives us the opportunity to > > > > use the ICU tools to customize/refresh/etc the data for Harmony each > > > > time the USA decide to introduce another daylight savings change ;-) > > > > > > exactly. > > > > > > > > > As expected, there are around 20 failures of harmony test. Many of > > > > > them are caused by data difference. I've listed all of them on harmony > > > > > wiki[2] and will raise them to icu soon. > > > > > > > > Ack. I believe that (pretty much) everyone gets the data from the CLDR, > > > > so I would expect any differences to be tests that have out of date > > > > assumptions about the data. > > > > > > Yes, I'll check it based on CLDR. > > > > > > > > > By applying this patch, we can get 4.2 mega bytes decrease in harmony > > > > > source code, including the svn data. > > > > > > > > ooh, say that again! Removing 4.2Mb of redundant source code is > > > > excellen > > t. > > > > > > > > > My proposal is to apply the patch on 5061 first and exclude these > > > > > tests which fail on different locale data. Then I'll take follow up > > > > > action to contact icu team and get solution to move them out. > > > > > Otherwise I'm afraid my fix will be outdated for waiting until icu > > > > > team get ready to fix the data. Do you have any objection? > > > > > > > > > > Later, I'll take care of the performance issue of this delegation. > > > > > > > > > > Thanks in advance for your comments > > > > > > > > Sounds like a good plan. > > > > > > > > Regards, > > > > Tim > > > > > > > > > > > > > -- > > > Tony Wu > > > China Software Development Lab, IBM > > > > > > > > > -- > > Tony Wu > > China Software Development Lab, IBM > > > > > -- Tony Wu China Software Development Lab, IBM
