Patchwork [2/6] buildhistory_analysis: fix duplicate PE/PV/PR in related fields

login
register
mail settings
Submitter Paul Eggleton
Date March 16, 2012, 7 p.m.
Message ID <2b1fc484fc04dcd1286f0b690cef5da879843d21.1331924409.git.paul.eggleton@linux.intel.com>
Download mbox | patch
Permalink /patch/23745/
State Accepted
Commit 406d025a15ff15a2edf39f00e0ea4e6b821b224a
Headers show

Comments

Paul Eggleton - March 16, 2012, 7 p.m.
Since PE, PV and PR appear in both the recipe history and package
history files these were showing up twice when they were added as
related fields to monitored changes. Only add them when the path is
exactly the same.

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 meta/lib/oe/buildhistory_analysis.py |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Patch

diff --git a/meta/lib/oe/buildhistory_analysis.py b/meta/lib/oe/buildhistory_analysis.py
index 4f3e635..d09911c 100644
--- a/meta/lib/oe/buildhistory_analysis.py
+++ b/meta/lib/oe/buildhistory_analysis.py
@@ -307,7 +307,7 @@  def process_changes(repopath, revision1, revision2 = 'HEAD', report_all = False)
                 if chg.path == chg2.path or os.path.dirname(chg.path) == chg2.path:
                     if chg2.fieldname in related_fields.get(chg.fieldname, []):
                         chg.related.append(chg2)
-                    elif chg.path.startswith('packages/') and chg2.fieldname in ['PE', 'PV', 'PR']:
+                    elif chg.path == chg2.path and chg.path.startswith('packages/') and chg2.fieldname in ['PE', 'PV', 'PR']:
                         chg.related.append(chg2)
 
     if report_all: