Re: commitinfo script to check policy, issue warnings (and/or abort commit)

[Alexandre Oliva <aoliva at redhat dot com>]

On May 14, 2001, Loren James Rittle <rittle@latour.rsch.comm.mot.com> wrote:

> for i in `basename $1`; do case $i in

basename is not portable.  I suggest `echo "$1" | sed 's,.*/,,'' instead.

>   boehm-gc)
>     echo "WARNING: Patches should be sent to boehm@acm.org, but it was"
>     echo "WARNING: acceptable to check them in the GCC CVS tree before"
>     echo "WARNING: getting them installed in the master tree."

I heard Hans wanted the GCC CVS tree to become the official repository
of boehm-gc.  I'm not sure this is already official.

> if (exit $message_emitted); then
>   echo "WARNING: See http://gcc.gnu.org/codingconventions.html for details."
> fi

I don't think we should prevent commits, but if check-ins can be
aborted at this time with ^C, I suggest telling the user so and giving
him/her like 30 seconds to decide whether to break the convention or
not.

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                  aoliva@{cygnus.com, redhat.com}
CS PhD student at IC-Unicamp        oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist    *Please* write to mailing lists, not to me