David,

Thanks for the information on this.  FYI I am on Subversion 1.4.2, and with the new white space command argument available in 1.4 and higher I was able to use it successfully with Codestriker.  Of course I would have achieved this much quicker if I had remembered to update the Perl bindings on my development system earlier than I did in my testing!  I edited, per your instructions the /lib/Codestriker/Repository/Subversion.pm to add the following in both the svn diff args commands as shown below.  I hope someone else finds this information useful.

@args = ();
push @args, 'diff';
push @args, '--non-interactive';
push @args, '--no-auth-cache';
push @args, '-x -w -b';
push @args, @{ $self->{userCmdLine} };
push @args, '-r';
push @args, $start_tag . ':' . $end_tag;
push @args, '--old';
push @args, $self->{repository_url};
push @args, $module_name;

Tim Casada

David Sitsky wrote:
Is there any way to change the configuration on codestriker to tweak the diff parameters for diff topics directly on top of source control systems?  We often format our code with a formatter and this sometimes causes lines to be marked as changed when they are really not.  I have no problem doing manual unified diff's but we really like the ease of doing reviews right on top of the version control systems.

At the moment, you'll need to modify the getDiff() method for your associated repository object and add in the appropriate command line switch.  The repository objects live in lib/Codestriker/Repository.

Cheers,
David



-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Codestriker-user mailing list
Codestriker-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/codestriker-user

Reply via email to