Last Comment Bug 696455 - Port |Bug 680373 - Link jemalloc to mozutils instead of mozalloc on Android| to comm-central
: Port |Bug 680373 - Link jemalloc to mozutils instead of mozalloc on Android| ...
Status: RESOLVED FIXED
:
Product: MailNews Core
Classification: Components
Component: Build Config (show other bugs)
: Trunk
: All All
: -- normal (vote)
: Thunderbird 11.0
Assigned To: Serge Gautherie (:sgautherie)
:
:
Mentors:
Depends on: 680373
Blocks: 696572
  Show dependency treegraph
 
Reported: 2011-10-21 12:24 PDT by Serge Gautherie (:sgautherie)
Modified: 2011-11-17 01:39 PST (History)
0 users
bugzillamozillaorg_serge_20140323: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
(Av1) configure.in: Sync' WRAP_MALLOC_* per m-c bug 680373 [Checked in: Comment 5] (2.34 KB, patch)
2011-10-21 12:28 PDT, Serge Gautherie (:sgautherie)
kairo: review+
Details | Diff | Splinter Review

Description Serge Gautherie (:sgautherie) 2011-10-21 12:24:50 PDT
Found while working on bug 694371.

Let's resync' c-c (step by step).
Comment 1 Serge Gautherie (:sgautherie) 2011-10-21 12:28:02 PDT
Created attachment 568738 [details] [diff] [review]
(Av1) configure.in: Sync' WRAP_MALLOC_* per m-c bug 680373
[Checked in: Comment 5]
Comment 2 Mark Banner (:standard8, limited time in Dec) 2011-10-25 06:51:31 PDT
Given this only affects android, which none of the c-c apps yet support, I have no idea why we'd even consider tracking this.
Comment 3 Mark Banner (:standard8, limited time in Dec) 2011-11-07 02:40:10 PST
Comment on attachment 568738 [details] [diff] [review]
(Av1) configure.in: Sync' WRAP_MALLOC_* per m-c bug 680373
[Checked in: Comment 5]

Sorry, but this looks like Android-only. As none of our c-c apps currently work with android, I'm not willing to spend time reviewing Android specific patches. I'd much rather have the time spent on bug 648979 which would stop the need for us for porting these bugs. You are welcome to ask other build config peers.
Comment 4 Robert Kaiser 2011-11-16 16:11:04 PST
Comment on attachment 568738 [details] [diff] [review]
(Av1) configure.in: Sync' WRAP_MALLOC_* per m-c bug 680373
[Checked in: Comment 5]

I don't really care about Android, as no c-c apps build on it anyhow, but I'd rather r+ it than let anyone else lose time over it.

Other than that, I agree with Mark's comment, though.
Comment 5 Serge Gautherie (:sgautherie) 2011-11-17 01:38:47 PST
Comment on attachment 568738 [details] [diff] [review]
(Av1) configure.in: Sync' WRAP_MALLOC_* per m-c bug 680373
[Checked in: Comment 5]

http://hg.mozilla.org/comm-central/rev/5698086670e5

Note You need to log in before you can comment on or make changes to this bug.