Remove MOZ_VALGRIND from comm-central

RESOLVED FIXED in Thunderbird 3.1b1

Status

defect
--
trivial
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

({fixed-seamonkey2.0.4})

Dependency tree / graph
Bug Flags:
in-testsuite -

Thunderbird Tracking Flags

(thunderbird3.0 .4-fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Assignee

Description

10 years ago
No description provided.
Flags: in-testsuite-
Assignee

Comment 1

10 years ago
Posted patch (Av1) Just copy it (obsolete) — Splinter Review
Attachment #425984 - Flags: review?(bugspam.Callek)
Attachment #425984 - Flags: approval-thunderbird3.0.2?
Attachment #425984 - Flags: approval-thunderbird3.0.2?
Attachment #425984 - Flags: review?(bugspam.Callek) → review+
Assignee

Comment 2

10 years ago
Morphing this bug: I forgot to check this one :-<
Blocks: CcConfCleanup
No longer blocks: C191ConfSync, C192ConfSync
No longer depends on: 475876
OS: Linux → All
Hardware: x86 → All
Summary: Port |Bug 475876 - TM: add hooks so Valgrind works with the JIT without requiring --smc-check=all| to comm-central → Remove MOZ_VALGRIND from comm-central
Assignee

Comment 3

10 years ago
Attachment #425984 - Attachment is obsolete: true
Attachment #426431 - Flags: review?(bugspam.Callek)
Attachment #426431 - Flags: review?(bugspam.Callek) → review+
Assignee

Comment 4

10 years ago
Comment on attachment 426431 [details] [diff] [review]
(Bv1) Just remove it
[Checkin: Comment 4 & 7]


http://hg.mozilla.org/comm-central/rev/ae2bc3f38c4a
Attachment #426431 - Attachment description: (Bv1) Just remove it → (Bv1) Just remove it [Checkin: Comment 4]
Assignee

Updated

10 years ago
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee

Comment 5

10 years ago
Comment on attachment 426431 [details] [diff] [review]
(Bv1) Just remove it
[Checkin: Comment 4 & 7]


"approval-thunderbird3.0.3=?":
Bug 475876 landed on m-1.9.1 too, Remove unsync'ed code.
Attachment #426431 - Flags: approval-thunderbird3.0.3?
Attachment #426431 - Flags: approval-thunderbird3.0.3? → approval-thunderbird3.0.4?
Attachment #426431 - Flags: approval-thunderbird3.0.4? → approval-thunderbird3.0.4+
Comment on attachment 426431 [details] [diff] [review]
(Bv1) Just remove it
[Checkin: Comment 4 & 7]

Ok, so the reason to take this is just that m-1.9.1 now has --enable-valgrind and we have --with-valgrind.
Assignee

Comment 7

9 years ago
Comment on attachment 426431 [details] [diff] [review]
(Bv1) Just remove it
[Checkin: Comment 4 & 7]


http://hg.mozilla.org/releases/comm-1.9.1/rev/ccf93af32ef2
Attachment #426431 - Attachment description: (Bv1) Just remove it [Checkin: Comment 4] → (Bv1) Just remove it [Checkin: Comment 4 & 7]
Verified on the basis this landed and hasn't broken the builds.

Updated

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