Closed
Bug 1031009
Opened 10 years ago
Closed 10 years ago
mach configure/build broken in c-c
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla33
People
(Reporter: clokep, Assigned: Paenglab)
References
Details
Attachments
(1 file)
1.92 KB,
patch
|
jcranmer
:
review+
|
Details | Diff | Splinter Review |
The "Reject builds with pymake" patch from bug 1027890 seems to have broken building with comm-central. My guess is that the usage of topsrcdir is wrong in this case.
The error one gets is: "Exception: Could not find a suitable make implementation." on Mac (or the corresponding message about using mozmake for Windows).
I don't think I have the knowledge to fix this, but if someone points me in the right direction I can give it a try!
Reporter | ||
Comment 1•10 years ago
|
||
Just to be explicit, I tested backing out rev 8aac3c8dc7bb from my copy of mozilla-central and I was able to run configure again. (And luckily it backs out cleanly.)
Comment 2•10 years ago
|
||
c-c's config/baseconfig.mk needs to get updates from at the very least bug 944650 and bug 1027890.
Assignee | ||
Comment 3•10 years ago
|
||
With this patch I can build again with mach. This is a simple copy-n-paste, I don't know what I'm doing here. Apologies for unneeded changes.
Attachment #8447012 -
Flags: review?(Pidgeot18)
Comment 4•10 years ago
|
||
Richard's patch *does* get Mach builds going again for me. I do have problems on a 'recurse to Libs' towards the end of the build however. Building with mozmake directly bypasses that problem, which seems to be specific to my build envir. (Win7, vc11 with the win8 sdk.)
Updated•10 years ago
|
Attachment #8447012 -
Flags: review?(Pidgeot18) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 5•10 years ago
|
||
Assignee: nobody → richard.marti
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•