[Top][All Lists]

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

Re: 'cvs watch on' patch (was: Changing passwords for CVS Users (pserver

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 is
wrong on feature. The new watch6-2 test fails - basically, the previous
watch add appears to be failing without reporting the error.  I've
attached a revised patch, with the failing tests (which, again, pass on

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.


reply via email to

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