Last Comment Bug 754065 - Port |Bug 744444 - delete CPP_PROG_LINK, purify/quantify targets| and |Bug 606145 part 1 - Properly link host programs written in C++|
: Port |Bug 744444 - delete CPP_PROG_LINK, purify/quantify targets| and |Bug 60...
Status: RESOLVED FIXED
:
Product: MailNews Core
Classification: Components
Component: Build Config (show other bugs)
: Trunk
: All All
: -- normal (vote)
: Thunderbird 15.0
Assigned To: Ian Neal
:
:
Mentors:
Depends on: 744444 747348
Blocks:
  Show dependency treegraph
 
Reported: 2012-05-10 14:54 PDT by Ian Neal
Modified: 2012-05-11 15:41 PDT (History)
2 users (show)
iann_bugzilla: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
fixed


Attachments
Port four patches as one [Checked in: Comment 2] (6.02 KB, patch)
2012-05-10 14:54 PDT, Ian Neal
bugspam.Callek: review+
Details | Diff | Splinter Review

Description Ian Neal 2012-05-10 14:54:02 PDT
Created attachment 622933 [details] [diff] [review]
Port four patches as one [Checked in: Comment 2]

To paraphrase bug 744444 comment 0:
Bug 747348 deleted two major users of CPP_PROG_LINK; the remaining ones are the purify and quantify targets.  I'm guessing those are not used much in this day and age, so they can be nuked.
This is a straight port of the 3 patches from bug 744444 with the addition of Bug 606145 part 1 (Properly link host programs written in C++)
Comment 1 Justin Wood (:Callek) 2012-05-11 11:11:03 PDT
Comment on attachment 622933 [details] [diff] [review]
Port four patches as one [Checked in: Comment 2]

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

Nit: remove |AC_SUBST(MOZ_QUANTIFY)| later in configure.in -- I know m-c didn't, but we don't have anything to subst for it.
Comment 2 Ian Neal 2012-05-11 15:40:19 PDT
Comment on attachment 622933 [details] [diff] [review]
Port four patches as one [Checked in: Comment 2]

Checked in with nit fixed:
http://hg.mozilla.org/comm-central/rev/8f22d9d7d692

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