Actions:
|
2012-08-08 21:19 AEST by Arthur Barrett - When you use 'cvs edit' on a file, to watch it or reserve it, there are two different types of 'error' response:
1)
cvs edit "421 read DAQ 1 channel vary range.vi"
cvsntsrv [editors-edit aborted]: Edit on file '421 read DAQ 1 channel vary range.vi' refused by server
2)
cvs edit "421 read DAQ 1 channel vary range.vi"
421 read DAQ 1 channel vary range.vi scott Wed Aug 8 10:04:52 2012 GMT
acerW64lap C:\Documents and Settings\abarrett\My Documents\scott-usadv\TestDev\ABTS\Test
cvs [edit aborted]: Files being edited!
Error 1) is a 'fatal' error and indicates either an ACL failure of reservation conflict
Error 2) is a 'non-fatal' error
The 'fatal' error is 'new' in CVSNT and the 'non-fatal' error is common with CVS and CVSNT. It was
decided that CVSNT should give this new error so that clients had the best chance of understanding that
the operation failed.
There is some argument that error 2 should be displayed before error 1, allowing clients to 'get what they
expect'. Since 'edit' could never 'fail' for CVS 1.x (because it was always concurrent and had no access
control except login failure) it is unlikely that the client would correctly understand error 1 to mean an
unsuccessful 'edit' anyway.
This could still be considered as an enhancement in the future, but since the current functionality works,
it's probably best left as it is. |