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:38 +0000 |
commit | b660a10e473ae49a1b3f9ec08be194dd4cf55afa (patch) | |
tree | c8f3e88da7e9426a467f2dca235d2c4b62dc0c51 /meta/lib/oe | |
parent | 97a91e81eaa4a893582161a13465775183669965 (diff) | |
download | poky-b660a10e473ae49a1b3f9ec08be194dd4cf55afa.tar.gz |
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.
(From OE-Core rev: 63fd76190f503660119dcc8efdcfc6fbff406c26)
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/oe')
-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): | |||
185 | adict = {} | 185 | adict = {} |
186 | for line in alines: | 186 | for line in alines: |
187 | splitv = [i.strip() for i in line.split('=',1)] | 187 | splitv = [i.strip() for i in line.split('=',1)] |
188 | if splitv.count > 1: | 188 | if len(splitv) > 1: |
189 | adict[splitv[0]] = splitv[1] | 189 | adict[splitv[0]] = splitv[1] |
190 | return adict | 190 | return adict |
191 | 191 | ||
@@ -231,7 +231,7 @@ def compare_file_lists(alines, blines): | |||
231 | filechanges.append(FileChange(path, FileChange.changetype_ownergroup, oldvalue, newvalue)) | 231 | filechanges.append(FileChange(path, FileChange.changetype_ownergroup, oldvalue, newvalue)) |
232 | # Check symlink target | 232 | # Check symlink target |
233 | if newsplitv[0][0] == 'l': | 233 | if newsplitv[0][0] == 'l': |
234 | if splitv.count > 3: | 234 | if len(splitv) > 3: |
235 | oldvalue = splitv[3] | 235 | oldvalue = splitv[3] |
236 | else: | 236 | else: |
237 | oldvalue = None | 237 | oldvalue = None |