(not currently active) Ted Mielczarek Reporter
	Description • 14 years ago

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.

	Gregory Szorc [:gps] Assignee
	Comment 1 • 11 years ago

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

	Gregory Szorc [:gps] Assignee
	Comment 2 • 11 years ago

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

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

	Gregory Szorc [:gps] Assignee
	Comment 3 • 11 years ago

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

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

	Michael Shal [:mshal]
	Comment 4 • 11 years ago

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

Looks good!

	Gregory Szorc [:gps] Assignee
	Comment 5 • 11 years ago

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

	Ryan VanderMeulen [:RyanVM]
	Comment 6 • 11 years ago

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