Hi.
Far be it for me, to contradict you on this, I'm no perl expert. but I can
only tell you my experience of trying to set this up earlier today.
If I call the script directly, like below, I get the following:
sunny-jim bin # ./subversion-post-commit.pl /var/svn/
svn.keergill.com/jetbyte 17
...
<title>Codestriker error</title>
...
<body bgcolor="white">
<p /><FONT COLOR='red'>Input parameter email has invalid value:
./subversion-post-commit.pl: `/usr/bin/svnlook info 17 -r /var/svn/
svn.keergill.com/jetbyte' failed with this
output:@keergill.com<output...@keergill.com></FONT><p
/>
</body>
...
Clearly from the error message its trying to apply the repository value to
the revision number parameter because its pulling off the parameters in the
reverse order to that expected by the code.
If I then reverse the parameters, I get the following
sunny-jim bin # ./subversion-post-commit.pl 17 /var/svn/
svn.keergill.com/jetbyte
...
<title>Codestriker error</title>
...
<body bgcolor="white">
<p /><FONT COLOR='red'>Topic state unrecognised</FONT><p />
</body>
...
So now I find that the call to CodestrikerClient::create_topic is failing
because the topic_state parameter is not initialised. If I add 'topic_state
=> "Open"' to the parameter list on the call to create_topic everything
works hunky dory (at least on first inspection, I haven't tried the script
in the subversion post-commit hook again yet.
Regards
Eugene Gill
2009/3/1 David Sitsky <david.sit...@gmail.com>
> Hi Eugene,
>
> I use this scripts at my work, so they work fine.
>
> The push and pop operators in perl work more in the metaphor of a
> stack, not a list. So pop removes the first element from the list, not
> the last.
>
> Cheers,
> David
>
> On Mon, Mar 2, 2009 at 2:10 AM, Eugene Gill <mi...@keergill.co.uk> wrote:
> > Hi.
> >
> > I'm just in the process of trying to set up Codestriker to automatically
> > generate reviews for SVN commits and am looking at this script. Has it
> > actually been tested?
> >
> > The following lines pop the command line arguments off from the end of
> the
> > list and therefore in the opposite order to that suggested in the comment
> > and the documentation.
> >
> > # First argument is the repository followed by the revision number.
> > my $repository = pop @ARGV;
> > my $revision = pop @ARGV;
> >
> > Apologies if I'm being somehow stupid, but it surprises me that this has
> > gone unspotted until now.
> >
> > Eugene
> >
> >
> >
> >
> >
> >
> >
> ------------------------------------------------------------------------------
> > Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco,
> CA
> > -OSBC tackles the biggest issue in open source: Open Sourcing the
> Enterprise
> > -Strategies to boost innovation and cut costs with open source
> participation
> > -Receive a $600 discount off the registration fee with the source code:
> SFAD
> > http://p.sf.net/sfu/XcvMzF8H
> > _______________________________________________
> > Codestriker-user mailing list
> > Codestriker-user@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/codestriker-user
> >
> >
>
------------------------------------------------------------------------------
Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA
-OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise
-Strategies to boost innovation and cut costs with open source participation
-Receive a $600 discount off the registration fee with the source code: SFAD
http://p.sf.net/sfu/XcvMzF8H
_______________________________________________
Codestriker-user mailing list
Codestriker-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/codestriker-user
