Bug 12742

Summary: buildhistory doesn't like files with whitespace
Product: [Build System, Metadata & Runtime] OE-Core Reporter: Ross Burton <ross.burton>
Component: devtools / tool chainAssignee: Paul Eggleton <bluelightning>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Medium+ CC: meta.mr.watcher, meta.watcher
Version: 2.5   
Target Milestone: 2.6 M2   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)

Description Ross Burton 2018-05-10 07:00:35 UTC
A recent version of gtk-doc added a file with a name something like this:

/usr/share/gtk-doc/html/templatate (devel).xml

However buildhistory writes the FILELIST by writing a whitespace separated list of filenames, without considering whitespace in filenames.

My ugly hack fix here to stop buildhistory-diff breaking (and thus my automated builds) is:

--- a/meta/classes/buildhistory.bbclass
+++ b/meta/classes/buildhistory.bbclass
-        filelist = list(dictval.keys())
+        filelist = [urllib.parse.quote(k) for k in dictval.keys()]

That doesn't handle making it transparent for loading though.
Comment 1 Paul Eggleton 2018-07-11 15:04:16 UTC
I just sent a patch for this, however for anyone else testing, FYI a reasonable test file would in fact be usr/lib/python2.7/site-packages/setuptools/script (dev).tmpl in python-setuptools rather than what was suggested above.
Comment 2 Ross Burton 2018-07-19 14:55:02 UTC
Fixed in oe-core c258379181a438cb01728d223b3d05e0ab205941.