diff options
author | Paul Eggleton <paul.eggleton@linux.intel.com> | 2012-03-20 16:06:28 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-03-21 14:16:36 +0000 |
commit | 33046605549501bed9ceff102614d86ba54a5891 (patch) | |
tree | 8acd6d1025b29faa402fd613c25b166a5e3b57a2 | |
parent | 2434a751e9684058786d4595cef3678d397f3c39 (diff) | |
download | openembedded-core-33046605549501bed9ceff102614d86ba54a5891.tar.gz openembedded-core-33046605549501bed9ceff102614d86ba54a5891.tar.bz2 openembedded-core-33046605549501bed9ceff102614d86ba54a5891.tar.xz openembedded-core-33046605549501bed9ceff102614d86ba54a5891.zip |
buildhistory_analysis: avoid printing PE/PV/PR more than once
Don't print PE/PV/PR changes as related field changes of related field
changes (i.e. only print them once at the top level).
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/lib/oe/buildhistory_analysis.py | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/meta/lib/oe/buildhistory_analysis.py b/meta/lib/oe/buildhistory_analysis.py index 4f55d9ab6..313416cf7 100644 --- a/meta/lib/oe/buildhistory_analysis.py +++ b/meta/lib/oe/buildhistory_analysis.py @@ -49,8 +49,8 @@ class ChangeRecord: def __str__(self): return self._str_internal(True) - def _str_internal(self, pathprefix): - if pathprefix: + def _str_internal(self, outer): + if outer: prefix = '%s: ' % self.path else: prefix = '' @@ -91,7 +91,7 @@ class ChangeRecord: percentchg = 100 out = '%s changed from %s to %s (%s%d%%)' % (self.fieldname, self.oldvalue or "''", self.newvalue or "''", '+' if percentchg > 0 else '', percentchg) elif self.fieldname in img_monitor_files: - if pathprefix: + if outer: prefix = 'Changes to %s ' % self.path out = '(%s):\n ' % self.fieldname if self.filechanges: @@ -107,6 +107,8 @@ class ChangeRecord: if self.related: for chg in self.related: + if not outer and chg.fieldname in ['PE', 'PV', 'PR']: + continue for line in chg._str_internal(False).splitlines(): out += '\n * %s' % line |