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 | |
| 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>
| -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 |
