Closed Bug 1061778 Opened 7 years ago Closed 7 years ago

Figure out how to express test addon xpi generation in moz.build

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 988938

People

(Reporter: ted, Unassigned)

References

(Blocks 1 open bug)

Details

Right now there's a Makefile that does this:
http://hg.mozilla.org/mozilla-central/annotate/c360f3d1c00d/toolkit/mozapps/extensions/test/Makefile.in#l11

Ideally we'd be able to express that in moz.build. Even building on top of my patches from bug 883954 there's no way to do it that's not super-verbose, you'd have to write something like:
GENERATED_FILES += [ '!/_tests/xpcshell/toolkit/mozapps/extensions/test/xpcshell/addons/whatever.xpi']
GENERATED_FILES['!/_tests/xpcshell/toolkit/mozapps/extensions/test/xpcshell/addons/whatever.xpi'].script = 'makexpi.py'

...which is actually worse.

Anyone have thoughts on this? Do we just punt and invent a TEST_XPIS variable?
how about saying the generated file is whatever.xpi and that its destination is '!/_tests/xpcshell/toolkit/mozapps/extensions/test/xpcshell/addons' ?
We do this in browser/experiments also. I think gps actually filed this already, but I can't find it right now.
I'm pretty sure I filed this bug as well. Coincidentally I've also been thinking about this problem because $(MAKE) XPI_NAME=foo hacks in Makefile.in are preventing build system optimizations that reduce no-op build times by 25%.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 988938
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.