Closed Bug 794988 Opened 12 years ago Closed 12 years ago

Enable profiling on Daily builds

Categories

(Thunderbird :: Build Config, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 18.0

People

(Reporter: mconley, Assigned: mconley)

Details

Attachments

(1 file, 2 obsolete files)

According to https://developer.mozilla.org/en-US/docs/Performance/Profiling_with_the_Built-in_Profiler#Running_the_profiler, we just need to add:

ac_add_options --enable-profiling

to the .mozconfigs for our Daily's.
You may want to stop stripping the builds of symbols as well if you plan on symbolicating on Mac.
Attached patch Stab-in-the-dark patch (obsolete) — Splinter Review
I'm not really savvy with our build config as it relates to our build machines, so I don't know if this patch does the job properly...

How's my driving?

Also, is there something I should be doing in here to stop stripping symbols, as Benoit suggests?
Assignee: nobody → mconley
Attachment #665521 - Flags: feedback?(mbanner)
Comment on attachment 665521 [details] [diff] [review]
Stab-in-the-dark patch

This is about right, but drop the comment and stick the line immediately under the signmar one, just like the Firefox mozconfigs (so that the diffs aren't too great).

We already have --disable-install-strip in the mac nightly mozconfigs, so we should be fine there.
Attachment #665521 - Flags: feedback?(mbanner) → feedback+
Attached patch Patch v1 (obsolete) — Splinter Review
Good point - that comment didn't really add much.

Done and done.
Attachment #665521 - Attachment is obsolete: true
Attachment #665592 - Flags: review?(mbanner)
Comment on attachment 665592 [details] [diff] [review]
Patch v1

Whoops, wrong patch file - one sec...
Attachment #665592 - Flags: review?(mbanner)
Attached patch Patch v1Splinter Review
Attachment #665592 - Attachment is obsolete: true
Comment on attachment 665593 [details] [diff] [review]
Patch v1

Ok, this is the real-deal here.
Attachment #665593 - Flags: review?(mbanner)
Comment on attachment 665593 [details] [diff] [review]
Patch v1

Excellent :-)

This can be landed with DONTBUILD in the comment, as it won't have any affect until the next nightlies.
Attachment #665593 - Flags: review?(mbanner) → review+
comm-central: https://hg.mozilla.org/comm-central/rev/85ae661d2a06
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: