Closed
Bug 777949
Opened 12 years ago
Closed 12 years ago
Make the editor Makefile.in's splendid
Categories
(Core :: DOM: Editor, defect)
Tracking
()
RESOLVED
FIXED
mozilla17
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
Details
Attachments
(1 file)
6.81 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
I have a patch which should make the Makefile.in's in editor/ BuildSplendid-friendly!
Assignee | ||
Comment 1•12 years ago
|
||
Comment 2•12 years ago
|
||
Comment on attachment 646358 [details] [diff] [review] Patch (v1) Review of attachment 646358 [details] [diff] [review]: ----------------------------------------------------------------- ::: editor/txmgr/Makefile.in @@ +18,1 @@ > endif We have a TEST_DIRS variable in rules.mk that is essentially: ifdef ENABLE_TESTS DIRS += TEST_DIRS endif Perhaps we should do something similar for test dirs in the tools tier? Better yet, perhaps we could just change all tests so they execute in the final tier (the tools tier)?
Assignee | ||
Comment 3•12 years ago
|
||
(In reply to comment #2) > Perhaps we should do something similar for test dirs in the tools tier? Better > yet, perhaps we could just change all tests so they execute in the final tier > (the tools tier)? That makes sense to me. You're not asking for me to do that in this bug, right? (I don't know how to do that!)
Comment 4•12 years ago
|
||
(In reply to Ehsan Akhgari [:ehsan] from comment #3) > (In reply to comment #2) > > Perhaps we should do something similar for test dirs in the tools tier? Better > > yet, perhaps we could just change all tests so they execute in the final tier > > (the tools tier)? > > That makes sense to me. You're not asking for me to do that in this bug, > right? (I don't know how to do that!) No, I'm not asking you to do it. I'm just planting a seed in the build peers' heads :)
Comment on attachment 646358 [details] [diff] [review] Patch (v1) Review of attachment 646358 [details] [diff] [review]: ----------------------------------------------------------------- ::: editor/Makefile.in @@ +11,5 @@ > > include $(DEPTH)/config/autoconf.mk > > +MODULE := editor > +DIRS := public idl txtsvc libeditor txmgr composer Can any of these run in parallel? Do txtsvc, libeditor, txmgr, and composer have dependencies on each other?
Attachment #646358 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 6•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d82321cde733 I'll try to parallelize these directories in another bug.
Target Milestone: --- → mozilla17
Assignee | ||
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d82321cde733
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•