Author: peter
Date: Thu Aug  6 18:19:30 2015
New Revision: 286377
URL: https://svnweb.freebsd.org/changeset/base/286377

Log:
  Check for some specific forms of broken commit templates.

Added:
  svnadmin/hooks/scripts/stomp_bad_formatting.pl   (contents, props changed)

Added: svnadmin/hooks/scripts/stomp_bad_formatting.pl
==============================================================================
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ svnadmin/hooks/scripts/stomp_bad_formatting.pl      Thu Aug  6 18:19:30 
2015        (r286377)
@@ -0,0 +1,54 @@
+#!/usr/bin/perl -w
+#
+# $FreeBSD$
+
+use strict;
+my $debug = 0;
+
+# $ svnlook changed /home/svnmirror/base -r 12348
+# UU  head/sbin/mountd/mountd.c
+# UU  head/usr.sbin/mountd/mountd.c
+# $ svnlook log /home/svnmirror/base -r 12348
+# Avoid bogus free() of a junk pointer.
+#
+# Detected by: phkmalloc
+# Submitted by:   g...@lemis.de (Greg Lehey)
+#
+# Except that when called to vette a commit, it'll be "-t txn", not "-r rev"
+
+
+my $repo = $ARGV[0];
+my $txn = $ARGV[1];
+
+my $log = "";
+
+open(LOG, "svnlook log $repo -t $txn |") || die "cannot open svnlook log: $!";
+foreach (<LOG>) {
+       print "$$: log: $_" if ($debug);
+       $log .= $_;
+}
+close(LOG);
+
+if (stomp_bad_formatting($log)) {
+       exit 1;
+}
+exit 0;
+
+# ============================================================
+# Look for a few specific mangled/broken template cases as a
+# stopgap for checking for a proper template.
+
+sub stomp_bad_formatting {
+       my ($log) = @_;
+       my $rv = 0;
+       if ($log =~ m|\n\nReviewers:[\t ]+|s) {
+               printf STDERR "**** Non-standard/badly formatted template - 
found 'Reviewers:' instead of 'Reviewed by:'.\n";
+               $rv = 1;
+       }
+       # There is really no need for this spam in log messages.
+       if ($log =~ m|\n\nSubscribers:[\t ]+|s) {
+               printf STDERR "**** Non-standard/badly formatted template - 
found 'Subscribers:'.\n";
+               $rv = 1;
+       }
+       $rv;
+}
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to