585016 - Invoke buildlist.py as a pymake native command

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[(not currently active) Ted Mielczarek]

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.

Comment 1

[Gregory Szorc [:gps]]

Attachment: Move buildlist.py to a mozbuild action

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

Comment 2

[Gregory Szorc [:gps]]

Attachment: Move buildlist.py to a mozbuild action

Turns out JarMaker.py was importing buildlist. I fixed the import and
removed some unused imports.

https://tbpl.mozilla.org/?tree=Try&rev=6cf1d3f0ca2e

Comment 3

[Gregory Szorc [:gps]]

Attachment: Move buildlist.py to a mozbuild action

Turns out you need a main() method in pymake commands.

https://tbpl.mozilla.org/?tree=Try&rev=99a26dff5939

Comment 4

[Michael Shal [:mshal]]

Comment on attachment 814316 [details] [diff] [review]
Move buildlist.py to a mozbuild action

Looks good!

Comment 5

[Gregory Szorc [:gps]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/05693539cfe6

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/05693539cfe6