summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPaul Eggleton <paul.eggleton@linux.intel.com>2012-03-20 16:06:28 +0000
committerRichard Purdie <richard.purdie@linuxfoundation.org>2012-03-21 14:16:36 +0000
commit33046605549501bed9ceff102614d86ba54a5891 (patch)
tree8acd6d1025b29faa402fd613c25b166a5e3b57a2
parent2434a751e9684058786d4595cef3678d397f3c39 (diff)
downloadopenembedded-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.py8
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