Remove leftover references to HAVE_CPP_MEMBER_TEMPLATES/HAVE_CPP_NUMERIC_LIMITS

RESOLVED FIXED in mozilla8

Status

()

Core
XPCOM
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: gaston, Assigned: gaston)

Tracking

Trunk
mozilla8
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Followup to bug 499262, those two HAVE_CPP should have been removed too in http://hg.mozilla.org/mozilla-central/rev/e5f0916b32d3. The HAVE_CPP macros are not set by anything in configure/buildsystem, and googling for them yields nothing but that mozilla code.
(Assignee)

Comment 1

6 years ago
Created attachment 548554 [details] [diff] [review]
Remove leftovers

Not even compile tested.. maybe needs a try run ?
Attachment #548554 - Flags: review?(benjamin)
(Assignee)

Updated

6 years ago
OS: OpenBSD → All
Hardware: x86 → All
Version: unspecified → Trunk
Comment on attachment 548554 [details] [diff] [review]
Remove leftovers

Yes, it should have a try run (build only) before landing.
Attachment #548554 - Flags: review?(benjamin) → review+
Assignee: nobody → landry
Status: NEW → ASSIGNED
(Assignee)

Comment 3

6 years ago
http://tbpl.mozilla.org/?tree=Try&rev=da747ad5fd37
(Assignee)

Comment 4

6 years ago
Try run finished, i can't make any sense from the red linux build opt nor the b-m test failure, i've asked for a rebuild but it doesnt seem to do anything...
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
(Assignee)

Comment 5

6 years ago
After asking for a bunch of single rebuilds, all is green. setting checkin-needed.
http://hg.mozilla.org/integration/mozilla-inbound/rev/9e084a7e9cee
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → mozilla8
http://hg.mozilla.org/mozilla-central/rev/9e084a7e9cee
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Comment 8

6 years ago
Thanks!
You need to log in before you can comment on or make changes to this bug.