Closed
Bug 975735
Opened 11 years ago
Closed 11 years ago
Port the LDFLAGS variables in netwerk/streamconv/test/Makefile.in to moz.build
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla30
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
References
Details
Attachments
(1 file)
1.13 KB,
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → ehsan
Blocks: xulinmozbuild
Assignee | ||
Updated•11 years ago
|
Attachment #8380200 -
Flags: review?(mshal)
Attachment #8380200 -
Flags: review?(mh+mozilla)
Attachment #8380200 -
Flags: review?(gps)
Comment 2•11 years ago
|
||
Comment on attachment 8380200 [details] [diff] [review]
Port the LDFLAGS variables in netwerk/streamconv/test/Makefile.in to moz.build
>diff --git a/netwerk/streamconv/test/moz.build b/netwerk/streamconv/test/moz.build
>index 2f65f17..e890c71 100644
>--- a/netwerk/streamconv/test/moz.build
>+++ b/netwerk/streamconv/test/moz.build
>@@ -11,8 +11,12 @@ UNIFIED_SOURCES += [
> 'TestStreamConv.cpp',
> ]
>
> FAIL_ON_WARNINGS = True
>
> if CONFIG['OS_ARCH'] == 'WINNT':
> for var in ('MOZ_NO_DEBUG_RTL', 'NGPREFS'):
> DEFINES[var] = True
>+ if CONFIG['GNU_CC']:
The Makefile had GNU_CXX here. I'm not sure if we would ever have GNU_CXX and not have GNU_CC, but this directory does only have .cpp files.
Attachment #8380200 -
Flags: review?(mshal)
Attachment #8380200 -
Flags: review?(mh+mozilla)
Attachment #8380200 -
Flags: review?(gps)
Attachment #8380200 -
Flags: review+
Assignee | ||
Comment 4•11 years ago
|
||
Assignee: nobody → ehsan
Comment 5•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
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
•