883538 - Install IPDL headers into dist/include

Status: NEW

(Core :: IPC, defect)

Description

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch v1

Instead of adding $(DEPTH)/ipc/ipdl/_ipdlheaders to LOCAL_INCLUDES every time we want to use them.

Comment 1

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Comment on attachment 763117 [details] [diff] [review]
Patch v1

Hrm, I think this actually makes us regenerate the headers and rebuild the world every top-level build... f? to see if you have a better idea

Comment 2

[Gregory Szorc [:gps]]

Comment on attachment 763117 [details] [diff] [review]
Patch v1

Review of attachment 763117 [details] [diff] [review]:
-----------------------------------------------------------------

If I am reading things correctly and my local build can be trusted, we don't currently copy ipdl headers into dist/include. So, this is a change in behavior and thus not warranted.

The ipdl files already have weird (read: bad) dependency foo going on. See bug 629668. After ipdl have moved into moz.build (bug 868536), we should completely overhaul the rules so things are sane. If you are going to touch ipdl make foo, I think I'd rather you start there. I have a build-ipdl patch in my patch queue that starts this work. Surely it's been bit rotted by other moz.build conversions.

Comment 3

[Michael Shal [:mshal]]

Note: if we end up installing ipdl headers into dist/include (or some similar already included path), we can probably remove the GeneratedInclude stuff from moz.build added in bug 928709.