Closed
Bug 520359
Opened 15 years ago
Closed 15 years ago
pymake fails to build Firefox
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(status1.9.2 beta5-fixed)
RESOLVED
FIXED
mozilla1.9.3a1
Tracking | Status | |
---|---|---|
status1.9.2 | --- | beta5-fixed |
People
(Reporter: Mitch, Assigned: ted)
References
Details
Attachments
(2 files)
2.10 KB,
patch
|
benjamin
:
review+
beltzner
:
approval1.9.2+
|
Details | Diff | Splinter Review |
773 bytes,
patch
|
wtc
:
review+
|
Details | Diff | Splinter Review |
Building Firefox from mozilla-central, using pymake currently fails. This patch fixes the non-NSPR parts. (Ted did all the work.)
Attachment #404411 -
Flags: review?(benjamin)
Reporter | ||
Comment 1•15 years ago
|
||
This patch fixes the NSPR part. (Again, Ted did all the work.)
Comment 2•15 years ago
|
||
Comment on attachment 404411 [details] [diff] [review] Patch v1.0 >diff --git a/config/config.mk b/config/config.mk > ifeq (,$(filter-out WINCE,$(OS_ARCH))) > NSINSTALL = $(CYGWIN_WRAPPER) nsinstall >-INSTALL = $(CYGWIN_WRAPPER) nsinstall >+INSTALL = $(CYGWIN_WRAPPER) nsinstall > endif Did you mean to use HOST_BIN_SUFFIX here, or why did you change this line at all?
Assignee | ||
Comment 3•15 years ago
|
||
I think I did initially change that, but then reverted it, but I guess I wound up removing some trailing whitespace there.
Assignee | ||
Updated•15 years ago
|
Attachment #404412 -
Flags: review?(wtc)
Assignee | ||
Comment 4•15 years ago
|
||
Comment on attachment 404412 [details] [diff] [review] Patch v1.0 (Part Deux) wtc, this is needed to get through a build with pymake, because `pwd` prints an MSYS path, and pymake wants Windows-style paths. $(CURDIR) has been available since GNU Make 3.77.
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → ted.mielczarek
Status: NEW → ASSIGNED
Comment 5•15 years ago
|
||
Comment on attachment 404412 [details] [diff] [review] Patch v1.0 (Part Deux) r=wtc. This construct is also used in mozilla/security/coreconf/rules.mk: http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/security/coreconf/rules.mk&rev=1.81&mark=394,400,403,409,413#388
Attachment #404412 -
Flags: review?(wtc) → review+
Assignee | ||
Comment 6•15 years ago
|
||
It looks like we're ok there, since they explicitly handle Win32 paths. We could probably simplify that whole block to use $(CURDIR), though.
Updated•15 years ago
|
Attachment #404411 -
Flags: review?(benjamin) → review+
Reporter | ||
Updated•15 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 7•15 years ago
|
||
Comment on attachment 404412 [details] [diff] [review] Patch v1.0 (Part Deux) Landed on NSPR trunk: Checking in config/rules.mk; /cvsroot/mozilla/nsprpub/config/rules.mk,v <-- rules.mk new revision: 3.73; previous revision: 3.72 done
Assignee | ||
Comment 8•15 years ago
|
||
Pushed the other patch to m-c: http://hg.mozilla.org/mozilla-central/rev/ccb86dc8e0da We'll need to get NSPR updated on trunk to pick up the other fix.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Updated•15 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 9•15 years ago
|
||
I updated NSPR on m-c, so this should all work again: http://hg.mozilla.org/mozilla-central/rev/d82fe5d73822
Updated•15 years ago
|
Attachment #404411 -
Flags: approval1.9.2?
Comment 10•15 years ago
|
||
Comment on attachment 404411 [details] [diff] [review] Patch v1.0 a192=beltzner
Attachment #404411 -
Flags: approval1.9.2? → approval1.9.2+
Comment 11•15 years ago
|
||
pushed http://hg.mozilla.org/releases/mozilla-1.9.2/rev/171e66b6364d
status1.9.2:
--- → final-fixed
Updated•14 years ago
|
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•