diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2015-06-18 15:14:16 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2015-06-23 11:57:25 +0100 |
commit | 86d30d756a60d181a95cf07041920a367a0cd0ba (patch) | |
tree | 950353e2e1cd1e7b812ff941b4d06076acd2c752 /meta/classes/report-error.bbclass | |
parent | f98c8490099a02c42306e1671579631a61c9df73 (diff) | |
download | poky-86d30d756a60d181a95cf07041920a367a0cd0ba.tar.gz |
meta: Add explict getVar param for (non) expansion
Rather than just use d.getVar(X), use the more explict d.getVar(X, False)
since at some point in the future, having the default of expansion would
be nice. This is the first step towards that.
This patch was mostly made using the command:
sed -e 's:\(getVar([^,()]*\)\s*):\1, False):g' -i `grep -ril getVar *`
(From OE-Core rev: ab7c1d239b122c8e549e8112c88fd46c9e2b061b)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes/report-error.bbclass')
-rw-r--r-- | meta/classes/report-error.bbclass | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/meta/classes/report-error.bbclass b/meta/classes/report-error.bbclass index 9edf2ceb31..c5aaaa8a11 100644 --- a/meta/classes/report-error.bbclass +++ b/meta/classes/report-error.bbclass | |||
@@ -29,11 +29,11 @@ python errorreport_handler () { | |||
29 | 29 | ||
30 | if isinstance(e, bb.event.BuildStarted): | 30 | if isinstance(e, bb.event.BuildStarted): |
31 | data = {} | 31 | data = {} |
32 | machine = e.data.getVar("MACHINE") | 32 | machine = e.data.getVar("MACHINE", False) |
33 | data['machine'] = machine | 33 | data['machine'] = machine |
34 | data['build_sys'] = e.data.getVar("BUILD_SYS", True) | 34 | data['build_sys'] = e.data.getVar("BUILD_SYS", True) |
35 | data['nativelsb'] = e.data.getVar("NATIVELSBSTRING") | 35 | data['nativelsb'] = e.data.getVar("NATIVELSBSTRING", False) |
36 | data['distro'] = e.data.getVar("DISTRO") | 36 | data['distro'] = e.data.getVar("DISTRO", False) |
37 | data['target_sys'] = e.data.getVar("TARGET_SYS", True) | 37 | data['target_sys'] = e.data.getVar("TARGET_SYS", True) |
38 | data['failures'] = [] | 38 | data['failures'] = [] |
39 | data['component'] = e.getPkgs()[0] | 39 | data['component'] = e.getPkgs()[0] |
@@ -64,7 +64,7 @@ python errorreport_handler () { | |||
64 | jsondata = json.loads(errorreport_getdata(e)) | 64 | jsondata = json.loads(errorreport_getdata(e)) |
65 | failures = jsondata['failures'] | 65 | failures = jsondata['failures'] |
66 | if(len(failures) > 0): | 66 | if(len(failures) > 0): |
67 | filename = "error_report_" + e.data.getVar("BUILDNAME")+".txt" | 67 | filename = "error_report_" + e.data.getVar("BUILDNAME", False)+".txt" |
68 | datafile = errorreport_savedata(e, jsondata, filename) | 68 | datafile = errorreport_savedata(e, jsondata, filename) |
69 | bb.note("The errors for this build are stored in %s\nYou can send the errors to a reports server by running:\n send-error-report %s [-s server]" % (datafile, datafile)) | 69 | bb.note("The errors for this build are stored in %s\nYou can send the errors to a reports server by running:\n send-error-report %s [-s server]" % (datafile, datafile)) |
70 | bb.note("The contents of these logs will be posted in public if you use the above command with the default server. Please ensure you remove any identifying or proprietary information when prompted before sending.") | 70 | bb.note("The contents of these logs will be posted in public if you use the above command with the default server. Please ensure you remove any identifying or proprietary information when prompted before sending.") |