[cvsnt] commitinfo script parameters bug in 2.0.9

[John Kinson]

Re: [cvsnt] commitinfo script parameters bug in 2.0.9Thanks Glen, that was
actually the first thing I tried before posting the mail, but nothing came
in over stdin.

However, your mail has jogged my memory that ActiveState Perl scripts that
are invoked via their extension do not receive stdin properly, so by
changing my commitinfo from

    ALL    C:/CVSRoot/Repo/CVSROOT/commitinfo.pl

to

    ALL    perl C:/CVSRoot/Repo/CVSROOT/commitinfo.pl

the problem was fixed, and the filenames do indeed get transferred over
stdin.

This interface change between the command line and stdin is bound to catch
people out though - it's a significant divergence from Cederqvist etc.,
although I understand the need for the change.  I guess I ought to read the
small print before upgrading...

Cheers

JK
--
John Kinson
http://www.yellowradio.com

----- Original Message -----
From: Glen Starrett
To: John Kinson ; cvsnt at cvsnt.org
Sent: Wednesday, September 10, 2003 8:51 PM
Subject: Re: [cvsnt] commitinfo script parameters bug in 2.0.9


On Wed, 10 Sep 2003 17:16:30 +0100, John Kinson <cvs at yellowradio.com>
wrote:

> After upgrading from CVSnt 2.0.4 to 2.0.9, I've noticed that the
> commitinfo
> hook argument passing seems to be broken.
>
[snip]
>
> Has the usage changed, is it me, or is this a bug?

I seem to recall that Tony said the behavior was going to change on one of
the scripts to make it consistent (and handle LARGE commits).  Look for the
filename info on STDIN, I bet you'll find it there.

Regards,

--
Glen Starrett