|
From: | Jim Hyslop |
Subject: | Re: 'cvs watch on' patch (was: Changing passwords for CVS Users (pserver protocol)] |
Date: | Wed, 23 Feb 2005 23:17:07 -0500 |
User-agent: | Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.5) Gecko/20041217 |
Derek Price wrote:
The patch does what you said it would on stable, but something else iswrong on feature. The new watch6-2 test fails - basically, the previouswatch add appears to be failing without reporting the error. I'veattached a revised patch, with the failing tests (which, again, pass onstable).
I've investigated, and watch6-2 appears to fail as a result of a different bug in feature. The commands "cvs watch [add/remove]" (with no files specified) no longer create the default entry in the fileattr file.
The test cases should pass on feature if you: sed 's/watch add/watch add .' sed 's/watch remove/watch remove .' I'll look into that other bug and see if I can track it down. -- Jim
[Prev in Thread] | Current Thread | [Next in Thread] |