Closed Bug 462730 Opened 16 years ago Closed 16 years ago

Building fails with -j19 option enabled

Categories

(Firefox Build System :: General, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: romaxa, Assigned: benjamin)

References

Details

Attachments

(2 files, 2 obsolete files)

Attached file Full build log (deleted) —
Mozconfig:
ac_add_options --enable-application=xulrunner
mk_add_options MOZ_OBJDIR=@TOPSRCDIR@/../obj-dir
mk_add_options MOZ_MAKE_FLAGS=-j19
ac_add_options --disable-javaxpcom

Fill build log in attachments
..............
No rule to make target `export'.  Stop.
make[6]: nsIWyciwygChannel.idl
*** [resources_export] Error 2
make[6]: *** Waiting for unfinished jobs....
creating content/xml/document/resources/Makefile
..............
creating content/xbl/test/Makefile
Creating .deps
make[5]: *** [document_export] Error 2
make[4]: *** [xml_export] Error 2
make[4]: *** Waiting for unfinished jobs....
nsIXMLContentBuilder.idl
creating content/mathml/content/src/Makefile
creating content/media/video/test/Makefile
..............
nsISelection.idl
Creating .deps
make[3]: *** [export_tier_gecko] Error 2
make[2]: *** [tier_gecko] Error 2
make[1]: *** [default] Error 2
make: *** [build] Error 2
Hm, it looks like your build started |make export| before it finished generating Makefiles. I guess we need an explicit dependency there somewhere.
Yeah, add %/Makefile as a dependency of the directory recursion rules.
Assignee: nobody → benjamin
Status: NEW → ASSIGNED
Attached patch add the right dependency on %/Makefile (obsolete) (deleted) — Splinter Review
romaxa: give this patch a try?

I had a dependency on the Makefiles, but it was not in the right place in the dependency graph, so the Makefiles could get generated in parallel with actually calling export on the dirs. Oops...
Assignee: benjamin → ted.mielczarek
Attachment #346043 - Flags: review?(benjamin)
Sunbird tinderbox triggers the same error (Bug 462839), at least I assume so.
Assignee: ted.mielczarek → benjamin
Attachment #346043 - Attachment is obsolete: true
Attachment #346055 - Flags: review?(ted.mielczarek)
Attachment #346043 - Flags: review?(benjamin)
Attachment #346055 - Flags: review?(ted.mielczarek) → review+
add phoniness
Attachment #346055 - Attachment is obsolete: true
Attachment #346057 - Flags: review?(ted.mielczarek)
Attachment #346057 - Flags: review?(ted.mielczarek) → review+
Pushed http://hg.mozilla.org/mozilla-central/rev/d38b03a1651b
and copied to js/src :-( http://hg.mozilla.org/mozilla-central/rev/c56b266cdb75
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Works fine now.
Status: RESOLVED → VERIFIED
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: