We should invoke buildlist.py as a pymake native command to avoid the overhead of spawning a new Python interpreter every time we run it.
Now that we have mozbuild "action" modules, it makes sense to move buildlist.py there. Patch should be relatively straightforward. https://tbpl.mozilla.org/?tree=Try&rev=73e125831a42
Turns out JarMaker.py was importing buildlist. I fixed the import and removed some unused imports. https://tbpl.mozilla.org/?tree=Try&rev=6cf1d3f0ca2e
Turns out you need a main() method in pymake commands. https://tbpl.mozilla.org/?tree=Try&rev=99a26dff5939
Attachment #814316 - Flags: review?(mshal)
Comment on attachment 814316 [details] [diff] [review] Move buildlist.py to a mozbuild action Looks good!
Attachment #814316 - Flags: review?(mshal) → review+
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.