Maven SCM

Errors during date parsing with cvs version 1.12.12.

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.0-beta-4
  • Fix Version/s: 1.0-rc1
  • Component/s: maven-scm-provider-cvs
  • Labels:
    None
  • Environment:
    Linux Suse 10
    CVS 1.12.12
  • Complexity:
    Intermediate
  • Number of attachments :
    0

Description

The changelog plugin used with cvs client 1.12.12, failed with the following error :
>> [ERROR] cvs [log aborted]: Can't parse date/time: `2007-02-20T12:15:51+0100'

The problem comes from the cvs client version 1.12.12 that doesn't seem any more to support the specified time format (yyyy-MM-dd'T'HH:mm:ssZ).

Tests made with changelog and cvs client version 1.11.6 are ok. Older versions of changelog, prior to issue SCM-177, and cvs client version 1.12.12 work also correctly.

The problem is solved by replacing the format "yyyy-MM-dd'T'HH:mm:ssZ" by this one "yyyy-MM-dd HH:mm:ssZ" in class AbstractCvsChangeLogCommand.

Issue Links

is depended upon by

Bug - A problem which impairs or prevents the functions of the product. MCHANGELOG-56 Date format not understood by CVS

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Emmanuel Venisse added a comment -

cvs 1.12.* isn't yet a stable release but a "feature" release and it isn't supported yet by Maven-SCM.
The date format is totally different in cvs 1.12 and we need to allow new formats.

I can't change the actual format by "yyyy-MM-dd HH:mm:ssZ" for all cvs client because lot of them doesn't support this format. I think I'll allow to change the format in cvs-settings.xml

Show
Emmanuel Venisse added a comment - cvs 1.12.* isn't yet a stable release but a "feature" release and it isn't supported yet by Maven-SCM. The date format is totally different in cvs 1.12 and we need to allow new formats. I can't change the actual format by "yyyy-MM-dd HH:mm:ssZ" for all cvs client because lot of them doesn't support this format. I think I'll allow to change the format in cvs-settings.xml
Hide
Permalink
Emmanuel Venisse added a comment -

yyyy-MM-dd HH:mm:ssZ format is used by default now as it work in cvs 1.11 too
If a cvs can't use this format, he'll can define an other one in cvs-settings.xml

Show
Emmanuel Venisse added a comment - yyyy-MM-dd HH:mm:ssZ format is used by default now as it work in cvs 1.11 too If a cvs can't use this format, he'll can define an other one in cvs-settings.xml

People

  • Assignee:
    Emmanuel Venisse
    Reporter:
    johann angeli
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: