Question on cvs acl options

[Abhilash Koneri]

Hi,

We are using cvs on a fairly large site with pserver option. We create
cvs users that are not system users and manage the module access using
a number of system users. However, this makes access management
somewhat difficult with growing number of projects and users. We have
tried to use cvs_acl script, but we found that it uses the user name
derived from the environment and not from the connection string passed
through the :pserver option. Is there an option (compile time or
runtime) that can be used to pass the user name in the pserver 
connection string to the pre-commit hooks ?  

We did modify the cvs source code of 1.11.20 to obtain the
functionality needed. (See the diff patch below). But we would like to
know if there is a standard option already available for this. If not,
would it be possible to include the same in the future builds as a
compile time or runtime option?  

Thanks,
Abhilash



address@hidden cvs-1.11.20]$ diff -r -C 5 src/commit.c  /home/akoneri/work/cvs-1.11.20/src/commit.c
*** src/commit.c        2005-03-17 10:32:05.000000000 -0600
--- /home/akoneri/work/cvs-1.11.20/src/commit.c 2005-08-01 22:59:13.000000000 -0500
***************
*** 1190,1199 ****
--- 1190,1206 ----
      }

      run_setup (filter);
      run_arg (repository);
      (void) walklist (saved_ulist, precommit_list_proc, NULL);
+     // :Change begin :
+     // Change to add the cvs user name into the arg list of the pre-commit
+     // proc
+     run_arg (CVS_Username);
+     // :Change end:
+     // Change to add the cvs user name into the arg list of the pre-commit
+     // proc
      return run_exec (RUN_TTY, RUN_TTY, RUN_TTY, RUN_NORMAL|RUN_REALLY);
  }



***************
*** 1224,1233 ****
--- 1231,1241 ----
        return err;

      /* run any pre-commit checks */
      if ((n = Parse_Info (CVSROOTADM_COMMITINFO, repos, precommit_proc, 1)) > 0)
      {
+     error (0,0, CVS_Username);
        error (0, 0, "Pre-commit check failed");
        err += n;
      }

      return err;