mirror of
https://git.yoctoproject.org/poky
synced 2026-01-29 21:08:42 +01:00
buildhistory_analysis: avoid noise due to reordering
Avoid noise in the output due to reordering of list variables (except for PACKAGES where we just report that the order changed). Recent changes to the buildhistory class itself will avoid this reordering from occurring but this allows us to examine the results before and after those changes. (From OE-Core rev: e23c5b01766602c9c86b0a7ba170fb3b1aceb658) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
committed by
Richard Purdie
parent
e155952b4d
commit
79eb4b8fdb
@@ -15,7 +15,8 @@ import git
|
||||
|
||||
|
||||
# How to display fields
|
||||
list_fields = ['DEPENDS', 'RDEPENDS', 'RRECOMMENDS', 'PACKAGES', 'FILES', 'FILELIST', 'USER_CLASSES', 'IMAGE_CLASSES', 'IMAGE_FEATURES', 'IMAGE_LINGUAS', 'IMAGE_INSTALL', 'BAD_RECOMMENDATIONS']
|
||||
list_fields = ['DEPENDS', 'RDEPENDS', 'RRECOMMENDS', 'FILES', 'FILELIST', 'USER_CLASSES', 'IMAGE_CLASSES', 'IMAGE_FEATURES', 'IMAGE_LINGUAS', 'IMAGE_INSTALL', 'BAD_RECOMMENDATIONS']
|
||||
list_order_fields = ['PACKAGES']
|
||||
numeric_fields = ['PKGSIZE', 'IMAGESIZE']
|
||||
# Fields to monitor
|
||||
monitor_fields = ['RDEPENDS', 'RRECOMMENDS', 'PACKAGES', 'FILELIST', 'PKGSIZE', 'IMAGESIZE']
|
||||
@@ -52,12 +53,15 @@ class ChangeRecord:
|
||||
else:
|
||||
prefix = ''
|
||||
|
||||
if self.fieldname in list_fields:
|
||||
if self.fieldname in list_fields or self.fieldname in list_order_fields:
|
||||
aitems = self.oldvalue.split()
|
||||
bitems = self.newvalue.split()
|
||||
removed = list(set(aitems) - set(bitems))
|
||||
added = list(set(bitems) - set(aitems))
|
||||
out = '%s:%s%s' % (self.fieldname, ' removed "%s"' % ' '.join(removed) if removed else '', ' added "%s"' % ' '.join(added) if added else '')
|
||||
if removed or added:
|
||||
out = '%s:%s%s' % (self.fieldname, ' removed "%s"' % ' '.join(removed) if removed else '', ' added "%s"' % ' '.join(added) if added else '')
|
||||
else:
|
||||
out = '%s changed order' % self.fieldname
|
||||
elif self.fieldname in numeric_fields:
|
||||
aval = int(self.oldvalue or 0)
|
||||
bval = int(self.newvalue or 0)
|
||||
@@ -237,6 +241,13 @@ def compare_dict_blobs(path, ablob, bblob, report_all):
|
||||
percentchg = 100
|
||||
if percentchg < monitor_numeric_threshold:
|
||||
continue
|
||||
elif (not report_all) and key in list_fields:
|
||||
alist = astr.split()
|
||||
alist.sort()
|
||||
blist = bstr.split()
|
||||
blist.sort()
|
||||
if ' '.join(alist) == ' '.join(blist):
|
||||
continue
|
||||
chg = ChangeRecord(path, key, astr, bstr, key in monitor_fields)
|
||||
changes.append(chg)
|
||||
return changes
|
||||
|
||||
Reference in New Issue
Block a user