[Buildbot] #2968: Include enough data to calculate blamelists
Buildbot trac
trac at buildbot.net
Fri Oct 24 14:48:11 UTC 2014
#2968: Include enough data to calculate blamelists
---------------------+---------------------
Reporter: dustin | Owner:
Type: defect | Status: new
Priority: critical | Milestone: 0.9.0
Version: master | Keywords: changes
---------------------+---------------------
{{{
10:41 < djmitche> so "what changes are included in a build" is actually a
function of *two* builds -- this one and the previous
10:41 < djmitche> think of "what changes are included in a build" as "git
log"
10:41 < djmitche> if you just give it one revision, it gives you all
changes from the first commit to that one
10:42 < djmitche> to get a limited set of commits, you need to give it two
revisions
10:43 < djmitche> what's missing right now in Buildbot is a 'parent'
pointer for changes
10:44 < djmitche> with that in place, the MailNotifier, etc. could
calculate the equivalent of 'git log last_build..this_build'
}}}
So this involves:
* adding a 'parents' field to changes (one-to-many)
* defining a utility to return the changes between two sourcestamps
* using that utility in status displays to define blamelists
Marked as a defect since this is re-implementing behavior that was
available in eight (only with more sense and less insanity).
--
Ticket URL: <http://trac.buildbot.net/ticket/2968>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation
More information about the bugs
mailing list