Closed
Bug 600236
Opened 14 years ago
Closed 14 years ago
Integrate valgrind into testing
Categories
(Tamarin Graveyard :: Virtual Machine, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: treilly, Assigned: brbaker)
References
Details
Attachments
(1 file)
4.23 KB,
patch
|
cpeyer
:
review+
|
Details | Diff | Splinter Review |
Maybe we could build one config with valgrind and run a test or two under valgrind in the smokes and then do a full valgrind run in deep test? would want to do mac and linux
Comment 1•14 years ago
|
||
Smokes are time limited; so we can have a valgrind smokes pass that won't make builds longer. Such a pass has the most value if the smokes are continualy updated to run the most recently broken things first (thats always been the plan but i suspect the smoke test list is getting stale).
Reporter | ||
Comment 2•14 years ago
|
||
I'd be happy if we added --enable-valgrind to one of the configs the smoke pass builds and run one simple test under valgrind. Extra time should be negligible.
Reporter | ||
Updated•14 years ago
|
Assignee: nobody → brbaker
Reporter | ||
Comment 3•14 years ago
|
||
Assigned to Brent, I'm pretty sure we did this.
Assignee | ||
Comment 4•14 years ago
|
||
- valgrind has been installed on mac and linux deep build slaves
- valgrind info has been added to buildbot setup wiki page
Attachment #510604 -
Flags: review?(cpeyer)
Updated•14 years ago
|
Attachment #510604 -
Flags: review?(cpeyer) → review+
Comment 5•14 years ago
|
||
changeset: 5905:6687de7bb4f5
user: Brent Baker <brbaker@adobe.com>
summary: Bug 600236: add release-debugger build and acceptance run through valgrind (r=cpeyer)
http://hg.mozilla.org/tamarin-redux/rev/6687de7bb4f5
Assignee | ||
Updated•14 years ago
|
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment 6•14 years ago
|
||
changeset: 5911:8035ea482acb
user: Brent Baker <brbaker@adobe.com>
summary: Bug 600236: verify that the build shell has the AVMFEAUTRE_VALGRIND feature enabled (r=brbaker)
http://hg.mozilla.org/tamarin-redux/rev/8035ea482acb
Assignee | ||
Updated•14 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•