1
0
mirror of https://git.yoctoproject.org/poky synced 2026-05-08 05:09:24 +00:00

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 <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Ross Burton
2016-07-12 10:23:18 +01:00
committed by Richard Purdie
parent 3ad542a85a
commit 65a0c27483
+5 -3
View File
@@ -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: