Re: Question on cvs acl options

[Abhilash Koneri]

Thanks! This worked!

On 10/11/05, Mark D. Baushke <address@hidden> wrote:

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi Abhilash,

Abhilash Koneri <address@hidden> writes:

> Thanks Mark. I tried the suggested procedure, but I got this error when
> trying to commit a file -

Hmmm.... my fault, the parsing of the commitinfo file does not actually
look at environment variables. You need to go one level indirect...

DEFAULT $CVSROOT/CVSROOT/cvs_acl_wrapper $CVSROOT/CVSROOT/cvs_acl

where cvs_acl_wrapper is a file that contains something like the following:

:
prog=$1; shift
exec "$prog" -u ${CVS_USER:-$USER} "$@"

> cvs commit: commitinfo:16: no such internal variable $CVS_USER
> cvs commit: Pre-commit check failed
> cvs [commit aborted]: correct above errors first!
>
> Is it the case that my cvs server does not have the "HAVE_PUTENV"
> functionality ? If so, can you tell me how I can enable the same? Is it a
> build option?

What is your platform? Most platforms have a putenv function these days.
Look in your config.h file for the HAVE_PUTENV to see what it says.

        -- Mark

>
> Thanks,
> Abhilash
>
>
> On 10/10/05, Mark D. Baushke <address@hidden> wrote:
> >
> > -----BEGIN PGP SIGNED MESSAGE-----
> > Hash: SHA1
> >
> > Abhilash Koneri <address@hidden> writes:
> >
> > > 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 ?
> >
> > You should be able to use $CVS_USER environment variable that is passed
> > to the administrative files.
> >
> > http://ximbiot.com/cvs/manual/cvs-1.11.21/cvs_18.html#SEC183
> >
> > Something like
> >
> > DEFAULT $CVSROOT/CVSROOT/cvs_acls -u $CVS_USER
> >
> > should do the trick for you.
> >
> > > We did modify the cvs source code of 1.11.20 to obtain the functionality
> > > needed.
> >
> > It should be available to you already without modification.
> >
> > > (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?
> >
> > If your server has a HAVE_PUTENV functionality, then CVS_USER should
> > get put into the environment for both :ext: and :pserver: mode.
> >
> > If you are using a :local: repsitory, then it will not be available.
> >
> > -- Mark
> > -----BEGIN PGP SIGNATURE-----
> > Version: GnuPG v1.2.3 (FreeBSD)
> >
> > iD8DBQFDSsXfCg7APGsDnFERAhW/AKCEOh+r2ctOtw+TJVyIy4rB/Dud2QCeJZoy
> > f6lZ6nsAu3NUBYMJcQ+vkq4=
> > =+kC3
> > -----END PGP SIGNATURE-----
> >
>
>
>
> --
>
> ---------------------------------------------------------------------------
> Gravity is a myth, the earth SUCKS!
> _______________________________________________
> Info-cvs mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/info-cvs
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (FreeBSD)

iD8DBQFDS2gzCg7APGsDnFERAqRFAJ0QaT6hOhfoV2sj3Ghqrm76S7C8OgCg55Z2
94s7VTlZC8xIQKdrQKVIDfQ=
=QkRa
-----END PGP SIGNATURE-----

-- 

---------------------------------------------------------------------------
Gravity is a myth, the earth SUCKS!