subversion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject svn commit: r1567519 - /subversion/trunk/tools/dist/
Date Wed, 12 Feb 2014 04:03:52 GMT
Author: breser
Date: Wed Feb 12 04:03:51 2014
New Revision: 1567519

Log: Avoid problems like Bert fixed in r1567239.

That problem was created by a revision not being prefixed by r.  So don't
require that.

Additional unreliability existed since the code used the r prefix to decide
if a line was a revision line and the revision list can wrap to the next line.
However, the logsummary line is difficult to distinguish from a wrapped revision
list.  So now we only treat a line as a revision list if all characters are used
up in calculating the revision list.  This also means if the first line used as
a revision list doesn't match the pattern then no revisions will be matched and
the line will error about having neither revisions or branch.  There is still
one ambiguous case.  If a revision list wraps and has some extra garbage in it,
it'll be treated as the log summary.  Can't really fix that without changing
the format.

* tools/dist/
  (parse_entry): Revision lines match all characters following the repeating
    pattern we're using.  The r prefix is optional.


Modified: subversion/trunk/tools/dist/
--- subversion/trunk/tools/dist/ (original)
+++ subversion/trunk/tools/dist/ Wed Feb 12 04:03:51 2014
@@ -356,9 +356,9 @@ sub parse_entry {
   $branch = sanitize_branch $1
     and shift
     if $_[0] =~ /^(\S*) branch$/ or $_[0] =~ m#branches/(\S+)#;
-  while ($_[0] =~ /^r/) {
+  while ($_[0] =~ /^(?:r?\d+[,; ]*)+$/) {
     my $sawrevnum = 0;
-    while ($_[0] =~ s/^r(\d+)(?:$|[,; ]+)//) {
+    while ($_[0] =~ s/^r?(\d+)(?:$|[,; ]+)//) {
       push @revisions, $1;

View raw message