svn commit: lorikeet r28 - hooks

[metze at samba.org]

Author: metze
Date: 2004-08-18 06:35:49 +0000 (Wed, 18 Aug 2004)
New Revision: 28

WebSVN: http://websvn.samba.org/websvn/changeset.php?rep=lorikeet&path=/&rev=28&nolog=1

Log:
- we now have 250 lines of inline diff
- we no longer have the repos name hardcoded in the WebSVN link

metze

Modified:
   hooks/commit-email.pl


Changeset:
Modified: hooks/commit-email.pl
===================================================================
--- hooks/commit-email.pl	2004-08-08 19:17:53 UTC (rev 27)
+++ hooks/commit-email.pl	2004-08-18 06:35:49 UTC (rev 28)
@@ -35,6 +35,7 @@
   }
 
 use strict;
+use File::Basename;
 use Carp;
 
 ######################################################################
@@ -54,7 +55,7 @@
 
 # Maximum number of lines to display in a diff.  If there are more
 # than this many lines, don't include a diff at all.
-my $max_difflines = 100;
+my $max_difflines = 250;
 
 # Since the path to svnlook depends upon the local installation
 # preferences, check that the required programs exist to insure that
@@ -337,11 +338,15 @@
 # Assembly of log message.
 
 # Put together the body of the log message.
+my $repname = basename($repos);
+my $websvn_link = "WebSVN: http://websvn.samba.org/websvn/changeset.php?rep=$repname\&path=/$commondir\&rev=$rev\&nolog=1\n";
 my @body;
 push(@body, "Author: $author\n");
 push(@body, "Date: $date\n");
 push(@body, "New Revision: $rev\n");
-push(@body, "WebSVN: http://websvn.samba.org/websvn/changeset.php?rep=lorikeet\&path=/$commondir\&rev=$rev\&nolog=1\n");
+push(@body, "\n");
+push(@body, $websvn_link);
+push(@body, "\n");
 push(@body, "Log:\n");
 push(@body, @log);
 if (@adds)
@@ -362,11 +367,12 @@
     push(@body, "Modified:\n");
     push(@body, map { "   $_\n" } @mods);
   }
-push(@body, "\nChangeset:\n");
+push(@body, "\n\nChangeset:\n");
 if ($#difflines <= $max_difflines) {
     push(@body, map { /[\r\n]+$/ ? $_ : "$_\n" } @difflines);
 } else {
-    push(@body, "Sorry the patch is to large, please use WebSVN to see it!\n");
+    push(@body, "Sorry, the patch is too large (".$#difflines." lines) to include; please use WebSVN to see it!\n");
+    push(@body, $websvn_link);
 };
        
 # Go through each project and see if there are any matches for this