Open Bug 1296708 Opened 8 years ago Updated 2 years ago

XPIDL invocation should be shared between backends

Categories

(Firefox Build System :: General, defect)

Product:
Firefox Build System
Component:
General
Type:
defect

Tracking

(Not tracked)

People

(Reporter: mshal, Unassigned)

References

Details

We still have some custom xpidl logic in config/makefiles/xpidl/Makefile.in that could be shared between backends. In particular, the xpidl-process.py invocation should be handled in moz.build somehow and given to the backend.

This is a followup from https://bugzilla.mozilla.org/show_bug.cgi?id=1293448#c10
Looking a little further, we'll probably also want to share the chrome.manifest and interfaces.manifest rules. The rest of the Makefile looks like it is specific to make's dependency processing.
Product: Core → Firefox Build System
Summary: XPIDL inovcation should be shared between backends → XPIDL invocation should be shared between backends
I believe that it should be possible to move at least some of the XPIDL processing into moz.build in a more reasonable way now, since the changes to how metadata is stored. The old concepts of .xpt file boundaries etc. are much less relevant today. We simply need the list of .xpt files to be built to be passed into a single python runtime execution, and it should be able to generate every relevant artifact.
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.