ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 16879] - CvsChangeLog attr "start" causes IllegalArgumentException
Date Thu, 20 Feb 2003 14:24:26 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=16879>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=16879

CvsChangeLog attr "start" causes IllegalArgumentException

conor@cortexebusiness.com.au changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED



------- Additional Comments From conor@cortexebusiness.com.au  2003-02-20 14:24 -------
I've committed a slightly different patch. Changing the signature of setStart()
is not a backward compatible change for anyone who has extended this task. While
that is likely to be the empty set, it was not really necessary. Instead I just
made setStateDate take the String argument.

Since this attribute is new, and not subject to backward compat, I made the
default date format the CVS default yyyy-MM-dd.

I introduced a new method determineDates which sets m_start from m_startDate, as
well as detecting any attempts to set the start date with two different methods.

Finally I did not apply the date format specified to the output date formatting
passed to CVS. This needs to be in a consistent format so CVS understands it.

I have not applied the doc and test patches yet. Can you check out if this code
works for you (you'll need to use "startdate" in place of "start", etc. If you
OK, I'll do the remaining work tomorrow.

Mime
View raw message