bug-cvs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: CVS update: MODIFIED: src ...


From: Larry Jones
Subject: Re: CVS update: MODIFIED: src ...
Date: Mon, 4 Nov 2002 14:18:35 -0500 (EST)

Derek Robert Price writes:
> 
> dotest rcs2-7 "${testcvs} -q update -p -D '100 months' file1" "head 
> revision"
> 
> Which seems to assume that eight years and four months from now (100 
> months) is later than 2004.08.31.01.01.01to get the "head revision" 
> result.  The archive file doesn't appear to be dynamically generated but 
> rather a static here-doc.  Therefore, doesn't any output other than 
> "head revision" mean that the checkout command itself is failing?

Yes, the checkout command is failing, but that's because the test is
deficient, not CVS.  I suppose it's a matter of perspective:  If you
naively add 100 months to October 31, 2002, you get February 31, 2011,
which is an invalid date.  That's what currently happens, so you get an
error that the specified date is invalid ("Can't parse date/time").  A
more sophisticated addition could be used to come up with a valid date,
but which one?  One could make a good argument for either March 1 or
March 3, but either one is apt to be surprising to some people.  Since
there's no single, obviously correct answer, I think producing an error
is the right thing to do.

-Larry Jones

Hmm... That might not be politic. -- Calvin




reply via email to

[Prev in Thread] Current Thread [Next in Thread]