Codestriker: collaborative code reviewer

Hi Guys,

I'm trying out codestriker and I've hit a bit of a wall. I can create a review topic 'by hand' using a diff I created myself but when it comes to supplying a tag/module set I either get error text or the page attempts to load and I end up with a blank page. I am using SVN.

my repo tree is structured like: http://localhost:80/svnrepo/Group1/Projects/Project1/

branches/branch_name/(source files)
tags/tag_name/(source files)
trunk/(source files)

I have a branch checked out at:

c:\svn\branch_name\( .svn folder is in here)

This branch is revision 3 and the trunk is revision 1.

What should I supply for module? The page ends up blank when I supply /Group1/Projects/Project1 but I'm new to subversion and I'm not 100% sure what I'm doing. I also tried supplying the absolute path to my checkout folder but it wasn't happy with that either.

When i navigate to c:\svn\branch_name and give:

> svn diff -r 1:3 > testdiff.txt

and then load testdiff.txt as part of the review topic it works just fine.

Any help would be much appreciated.

Thanks,

Sam

This all comes down to configuring your repository object right. If you can send me the output of your diff, that would be great, but you need to specify your svn path fully as something like this:

/something/here/svnrepo/Group1/Projects/Project1/branches/branch_name

See the codestriker.conf for some svn examples. It needs to match what appears in your manual diff file.

Then module would be just testdiff.txt, start_tag would be 1, end_tag would be 3.

Cheers,
David

when navigating to my branch checkout at

c:\svn\Magnus_b_5_0_6_0_TestReview

and executing:

> svn diff -r 1:3 > testdiff.txt

the output is:

-------------------------------------------------------------------------------------------------------
Index: Source/SMARTEvent.c
===================================================================
--- Source/SMARTEvent.c (.../trunk) (revision 1)
+++ Source/SMARTEvent.c (.../branches/Magnus_b_5_0_6_0_TestReview) (revision 3)
@@ -4,6 +4,10 @@
* SMART Technologies Inc.
********************************************************************/

+// changed this line for testing purposes with codestriker
+#ifndef SEVEN
+#define SEVEN 7
+#endif
/*C**************************************************************************
* NAME: SMARTEvent.c
* This file contains all the event handler functions for the SC9.
Index: Source/magnus.c
===================================================================
--- Source/magnus.c (.../trunk) (revision 1)
+++ Source/magnus.c (.../branches/Magnus_b_5_0_6_0_TestReview) (revision 3)
@@ -4,6 +4,11 @@
* SMART Technologies Inc.
********************************************************************/

+
+// changed this line for testing purposes with codestriker
+#ifndef SEVEN
+#define SEVEN 7
+#endif
/********************************************************************
* File Name: magnus.c
* Description:
---------------------------------------------------------------------------------------------------

I'm not sure I understand this. Are you saying the module is simply the name of the diff output if you want a diff output file?

I see my problem now. I was trying to compare my branch checkout to the trunk.

When I use it for changes in the same branch it works fine.