From 65a0c274833ac84953458547dd01d70bd881b6bb Mon Sep 17 00:00:00 2001 From: Ross Burton Date: Tue, 12 Jul 2016 10:23:18 +0100 Subject: report-error: replace build paths with markers not whitespace To remove potentially personal information and reduce irrelevant noise when searching for similar reports the error reporting class removes ${TOPDIR} from the logs. Whilst this is valid intention, the replacement of ' ' results in potentially confusing logs as it appears that builds are happening in /tmp, or whitespace can appear in places where it isn't allowed which can look like a bug. Solve both of these by replacing the value of TOPDIR with the literal string TOPDIR. Also replace TMPDIR after TOPDIR, as it's not uncommon to have TMPDIR somewhere other than directly under TOPDIR. (From OE-Core rev: 95794e261628f83ddab0aa7b8bafb6409cc9deb5) Signed-off-by: Ross Burton Signed-off-by: Richard Purdie --- meta/classes/report-error.bbclass | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'meta/classes/report-error.bbclass') diff --git a/meta/classes/report-error.bbclass b/meta/classes/report-error.bbclass index 5cbf8f07fd..c8c15bdafa 100644 --- a/meta/classes/report-error.bbclass +++ b/meta/classes/report-error.bbclass @@ -59,9 +59,11 @@ python errorreport_handler () { logFile = codecs.open(log, 'r', 'utf-8') logdata = logFile.read() - topdir = e.data.getVar('TOPDIR', True) - if topdir: - logdata = logdata.replace(topdir, ' ') + # Replace host-specific paths so the logs are cleaner + for d in ("TOPDIR", "TMPDIR"): + s = e.data.getVar(d, True) + if s: + logdata = logdata.replace(s, d) logFile.close() except: -- cgit v1.2.3-54-g00ecf