diff options
| author | Paul Eggleton <paul.eggleton@linux.intel.com> | 2012-12-06 11:11:51 +0000 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-12-06 13:51:32 +0000 |
| commit | 63fd76190f503660119dcc8efdcfc6fbff406c26 (patch) | |
| tree | 851ca1032af498a6fbcf7be5ee9efc26e14529ec | |
| parent | 6d86690330f0d43839b904fced4b4b02cb27b8c6 (diff) | |
| download | openembedded-core-63fd76190f503660119dcc8efdcfc6fbff406c26.tar.gz openembedded-core-63fd76190f503660119dcc8efdcfc6fbff406c26.tar.bz2 openembedded-core-63fd76190f503660119dcc8efdcfc6fbff406c26.zip | |
buildhistory_analysis: fix broken list length checks
Fix erroneous use of .count instead of len(), which unfortunately is not
reported by Python as an error in a numeric comparison.
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 | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/meta/lib/oe/buildhistory_analysis.py b/meta/lib/oe/buildhistory_analysis.py index ad57f00289..7b5ee4551a 100644 --- a/meta/lib/oe/buildhistory_analysis.py +++ b/meta/lib/oe/buildhistory_analysis.py @@ -185,7 +185,7 @@ def blob_to_dict(blob): adict = {} for line in alines: splitv = [i.strip() for i in line.split('=',1)] - if splitv.count > 1: + if len(splitv) > 1: adict[splitv[0]] = splitv[1] return adict @@ -231,7 +231,7 @@ def compare_file_lists(alines, blines): filechanges.append(FileChange(path, FileChange.changetype_ownergroup, oldvalue, newvalue)) # Check symlink target if newsplitv[0][0] == 'l': - if splitv.count > 3: + if len(splitv) > 3: oldvalue = splitv[3] else: oldvalue = None |