[cvsspam-devel] CVSspam for subversion

[Elan Ruusamäe]

On Thursday 09 March 2006 05:55:14 Joe Eiler wrote:
> The second "svn_support" patch adds the stuff related to subversion.

why the patch adds this $shortrepo?

it breaks my setup (worked before applying this patchset) as the leading svn 
repository root name (basename actually)  is added to the path, like it was 
svn path, making web links invalid.

perhaps it could be added as optionally, maybe some new hook<>cvsspam.rb 
protocol, like "#R {$shortrepo}" (not sure what those #M, etc do)

or simpliest would just be option to svn_post_commit_hook.rb argument?

 def process_modified_binary_diff(out, lines, path)
-  prev_rev= "unknown"
-  next_rev= "unknown"
+  prev_rev= $revision-1
+  next_rev= $revision
   out.puts "#V #{prev_rev},#{next_rev}"
-  out.puts "#M #{path}"
+  out.puts "#M #{$shortrepo}/#{path}"
   out.puts "#U diff x x"
   out.puts "#U Binary files x and y differ"

-- 
glen