Closed Bug 916257 Opened 7 years ago Closed 7 years ago

modernize ipdlsrcs.mk writing

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla27

People

(Reporter: froydnj, Unassigned)

Details

Attachments

(3 files, 2 obsolete files)

I started refactoring ipdlsrcs.mk writing so that I could build ipc/ipdl in
unified mode per bug 907789.  I ran into some problems, which need to be
solved elsewhere, b ut I thought I would at least throw the patches up for
review now, since they're all finished.
We have better ways to do things nowadays.  Trivial.
Attachment #804627 - Flags: review?(gps)
This patch is probably misnamed; it's mostly here to separate the logic for
determining what files we're generating from a single ipdl file from the logic
for actually writing the necessary definitions.
Attachment #804628 - Flags: review?(gps)
Maybe not strictly necessary, but if pymake is happier with a single statement
rather than a bunch of variable appends, a small efficiency improvement.
Attachment #804629 - Flags: review?(gps)
Attachment #804627 - Flags: review?(gps) → review+
Attachment #804628 - Flags: review?(gps) → review+
Comment on attachment 804629 [details] [diff] [review]
part 3 - write ALL_IPDLSRCS and CPPSRCS in one go

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

Huzzah!
Attachment #804629 - Flags: review?(gps) → review+
Sorry, I backed this out on inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/dbb0d3946a92

because of this build error:
NameError: global name 'mozmakeutil' is not defined
https://tbpl.mozilla.org/php/getParsedLog.php?id=27853917&tree=Mozilla-Inbound
https://hg.mozilla.org/integration/mozilla-inbound/rev/a8da12b9e73f
https://hg.mozilla.org/integration/mozilla-inbound/rev/e9a59bc50ffb
https://hg.mozilla.org/integration/mozilla-inbound/rev/e068176c4814

Actually |import|'d all the things I was supposed to this time, rather than relying on previous patches in my queue to provide them.
Ah, the tests, the tests.  Must fix the tests.

I think the blank lines technically from from the mozbuild.makeutil conversion,
but I am being lazy and not splitting the test changes up amongst all three
patches...
Attachment #808595 - Flags: review?(gps)
More test changes.  Wasn't quite sure what to do about the overly long lines; please
yell if the chosen path is not acceptable.
Attachment #804629 - Attachment is obsolete: true
Attachment #808596 - Flags: review?(gps)
Attachment #804628 - Attachment is obsolete: true
Attachment #808595 - Flags: review?(gps) → review+
Comment on attachment 808596 [details] [diff] [review]
part 3 - write ALL_IPDLSRCS and CPPSRCS in one go

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

I am slightly scared by the frequency you use itertools. The force is strong with you.
Attachment #808596 - Flags: review?(gps) → review+
(In reply to Gregory Szorc [:gps] from comment #11)
> I am slightly scared by the frequency you use itertools. The force is strong
> with you.

\o/
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.