Define MOZ_CODE_COVERAGE for the entire tree instead of individually in several moz.build files

RESOLVED FIXED in Firefox 56

Status

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: marco, Assigned: marco)

Tracking

(Blocks 1 bug)

Version 3
mozilla56
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

We're currently adding MOZ_CODE_COVERAGE to DEFINES in several moz.build files throughout the tree, it would be better to just define it for the entire tree.
Posted patch Patch (obsolete) — Splinter Review
Assignee: nobody → mcastelluccio
Status: NEW → ASSIGNED
Attachment #8886195 - Flags: review?(jmaher)
Posted patch PatchSplinter Review
I had to remove MOZ_CODE_COVERAGE from old-configure.in, otherwise it would have been added to config twice.

Here's a green try build: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6549b982544def517db9656cd4fe1c5d92ef4f20

The mozinfo.json contains correct info ("ccov" is true): https://public-artifacts.taskcluster.net/V4zpl-GWS4alC7XLuZfIXg/0/public/build/target.mozinfo.json.
Attachment #8886195 - Attachment is obsolete: true
Attachment #8886195 - Flags: review?(jmaher)
Attachment #8886296 - Flags: review?(jmaher)
Comment on attachment 8886296 [details] [diff] [review]
Patch

Review of attachment 8886296 [details] [diff] [review]:
-----------------------------------------------------------------

thanks!
Attachment #8886296 - Flags: review?(jmaher) → review+
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/32aa8aa9330b
Define MOZ_CODE_COVERAGE for the entire tree. r=jmaher
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/32aa8aa9330b
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Duplicate of this bug: 1380663
You need to log in before you can comment on or make changes to this bug.