696455 - Port |Bug 680373 - Link jemalloc to mozutils instead of mozalloc on Android| to comm-central

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Serge Gautherie (:sgautherie)]

Found while working on bug 694371.

Let's resync' c-c (step by step).

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) configure.in: Sync' WRAP_MALLOC_* per m-c bug 680373 [Checked in: Comment 5]

Comment 2

[Mark Banner (:standard8)]

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)]

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]

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)]

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