Summary: | cvschangelog throes NullPointerException with some CVS server | ||
---|---|---|---|
Product: | Ant | Reporter: | Dmitry <zakharov> |
Component: | Core tasks | Assignee: | Ant Notifications List <notifications> |
Status: | RESOLVED DUPLICATE | ||
Severity: | normal | CC: | imadam |
Priority: | P2 | ||
Version: | 1.6.2 | ||
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | Windows 2000 |
Description
Dmitry
2004-12-08 12:07:02 UTC
Just curious. How do you have that date format ? Did you patch your cvs server to have ISO8601 date ? The cvs server was installed by our customer. So I don't know about the patach mentioned. Also I'd like to add that the date which is tried to parse doesn't contain timezone information. I mean that our cvs server returns 2004-12-07 10:38:04 +0000, but only String 2004-12-07 10:38:04 is passed to parse into SimpleDateFormat. I think SimpleDateFormat will use default client time zone, which is not +0000 in our case. We encountered this also after moveing from an old CVS server to a new one. New CVS versions supply ISO date format, eg from http://jira.public.thoughtworks.org/browse/CC-38 In cvs 1.12.9, the output format of the cvs log command was changed. In particular, the date: line now uses "-" as a date separator instead of "/", and adds a new field ("+0000"). The code at issue is http://cvs.apache.org/viewcvs.cgi/ant/src/main/org/apache/tools/ant/taskdefs/cvslib/ChangeLogParser.java?rev=1.29&view=markup class ChangeLogParser { ... /** input format for dates read in from cvs log */ private static final SimpleDateFormat INPUT_DATE = new SimpleDateFormat("yyyy/MM/dd HH:mm:ss"); ... } Also BUG 30576 looks like a duplicate of this one (or vice versa.) |