commit_prep and cvs_acls question

[yancheng . cheok]

address@hidden

Hello all, I wish to try out Hampton's cool perl cvs_acls script which
is found in cvs-1.11.21/contrib folder.

>From the manual, it states that

If you are using client-side userIds to restrict access to your
repository, make sure that they are in this order since the commit_prep
script is required in order to pass the $USER parameter.

So, in my commitinfo files, here are the lines

ALL $CVSROOT/CVSROOT/commit_prep
ALL $CVSROOT/CVSROOT/cvs_acls -d -u $USER

And here are the cvsacl

deny||CVSROOT/commitinfo CVSROOT/cvsacl
allow|yccheok|CVSROOT/commitinfo CVSROOT/cvsacl

However, when I login as yccheok and try to commit "commitinfo", I get
the following error:

**** Access allowed: Sufficient authority for commit.
30797 ==== $exit_val = 0
cvs commit: Pre-commit check failed
cvs [commit aborted]: correct above errors first!


So, I edit the commitinfo by hand with the following lines

## ALL $CVSROOT/CVSROOT/commit_prep
ALL $CVSROOT/CVSROOT/cvs_acls -d -u $USER

**** Access allowed: Sufficient authority for commit.
30802 ==== $exit_val = 0
Checking in commitinfo;
/cvsroot/CVSROOT/commitinfo,v  <--  commitinfo
new revision: 1.4; previous revision: 1.3

It seems that once I didn't use commit_prep script, everything goes
fine.

May I know why do we actually need commit_prep script?

Thank you very much!

cheok