[Buildbot-devel] Patch for [1398174] svn_buildbot fails to process property changes correctly

[Olivier Bonnet]

Here's a small patch fixing [ 1398174 ] svn_buildbot fails to process
property changes correctly [1]

The patch is dead simple: when doing a 'svnlook changed' (or a 'svn
status'), the 6 first columns can be characters giving various
information as described in the SVN book [2].
The svn_buildbot.py script was only removing the first one.

Olivier


[1] https://sourceforge.net/tracker/index.php?func=detail&aid=1398174&group_id=73177&atid=537001
[2] http://svnbook.red-bean.com/nightly/en/svn.tour.cycle.html#svn.tour.cycle.examine.status

--- orig/contrib/svn_buildbot.py
+++ mod/contrib/svn_buildbot.py
@@ -145,7 +145,8 @@
         changed = commands.getoutput('svnlook changed %s "%s"' % (rev_arg,
                                                                   repo)
                                      ).split('\n')
-        changed = [x[1:].strip() for x in changed]
+        # the first 6 columns can contain status information
+        changed = [x[6:].strip() for x in changed]

         message = commands.getoutput('svnlook log %s "%s"' % (rev_arg, repo))
         who = commands.getoutput('svnlook author %s "%s"' % (rev_arg, repo))