On line 288, there is the following code: if options.cachedir is not None: if not os.path.isdir(options.cachedir): os.mkdir(options.cachedir) but it's missing the "import os" part. C.f: http://hg.mozilla.org/mozilla-central/file/15bfad783467/xpcom/idl-parser/typelib.py
Also, relating code is also missing the "import sys" too (see the lines below 288).
Created attachment 565567 [details] [diff] [review] Patch to import the missing modules
Comment on attachment 565567 [details] [diff] [review] Patch to import the missing modules Which branches do we need this on?
This would be needed on Mozilla 9.0 and higher, i.e. at present that is the following repos: * mozilla-aurora * mozilla-central
Comment on attachment 565567 [details] [diff] [review] Patch to import the missing modules Build tool change, 0 risk, unbreaks third parties.
http://hg.mozilla.org/mozilla-central/rev/1318d1bbc15a http://hg.mozilla.org/releases/mozilla-aurora/rev/51d973c83798 Thanks for the patch Todd.
Marking qa- as this is not something QA needs to verify. Please change to qa+ if this is not the case.