Re: add hook question (was Re: Problem with importing third-party sources and adding/committing changes)

[Mark D. Baushke]

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

Greg A. Woods <address@hidden> writes:

> There is no trigger for "cvs rm" and there MUST _NOT_ be.
>
> There is no trigger for "cvs add" and there MUST _NOT_ be.

Please defend your position for 'MUST NOT' as it seems an entirely
reasonable suggestion that additional triggers might wish to be imposed
by a repository administrator.

Today, there is no direct trigger for 'cvs rm'. I do not have strong
feelings about adding one because existing clients do not yet talk to
the server for this operation which only becomes visible when the user
does a commit and a user could always work around the prohibition of
doing a 'cvs rm' by truncating the file to zero length and committing
that which will not matter most of the time.

Today, there is no direct trigger for 'cvs add', however, a 'cvs add' of
a directory at present is not possible to stop and there are some
administrators who would like to impose standards on directory names in
the repository. For example, some folks would like to impose the 'no
spaces allowed in directory names' to make it possible to add modules
entries for all possible directories.

This same rational is also true for 'cvs import' which allows users to
specify lots of directories and files that only the loginfo script sees
after the fact with no checking to see if the vendor tag, version tag or
directory names or file names match those allowed by the policy of the
administrator. More than one user has requested a trigger for 'cvs
import' that gets called before everything just gets dumped into the
loginfo trigger. Others have suggested that a series of commitinfo and
taginfo may be suitable as an alternative to an importinfo trigger.

        -- Mark
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (FreeBSD)

iD8DBQFBnO3s3x41pRYZE/gRAhdJAKCCBNbUqYkAbG3Wb7UpXLEVVUfMHgCdHHU6
szKXthS/vS5XcP/ExzKlXFc=
=U1Cb
-----END PGP SIGNATURE-----