Closed
Bug 544117
Opened 16 years ago
Closed 16 years ago
Remove avm selftest compile steps now that selftest is enabled for all configs.
Categories
(Tamarin Graveyard :: Tools, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: cpeyer, Assigned: cpeyer)
Details
Attachments
(2 files)
|
26.31 KB,
patch
|
brbaker
:
review+
|
Details | Diff | Splinter Review |
|
5.20 KB,
patch
|
cpeyer
:
review+
|
Details | Diff | Splinter Review |
Lars enabled selftest for all configs with TR r3731. We can now remove the explict compile step for selftest builds and replace it with the release build (we were compiling a release selftest build in buildbot).
| Assignee | ||
Comment 1•16 years ago
|
||
I did not remove the shell_selftest env variable in case we needed to have an explicit selftest build again in the future (though it was removed for the coverage build). shell_selftest is now just pointed to the release build.
Attachment #425084 -
Flags: review?(brbaker)
Updated•16 years ago
|
Attachment #425084 -
Flags: review?(brbaker) → review+
Comment 2•16 years ago
|
||
Comment on attachment 425084 [details] [diff] [review]
remove selftest compilation and replace with release build
Removed more references to uploading and build checks for selftest and pushed buildslave script changes as:
tr-argo: 3659:6a283ef36736
tr: 3741:6a283ef36736
Updated buildbot configuration as changeset 3743:b3780dcf107d
Updated•16 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Comment 3•16 years ago
|
||
reopening, need to do a little more work to the build system to make this clean
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 4•16 years ago
|
||
Update the size report scripts to disable the selftests from being compiled
Attachment #425208 -
Flags: review?(cpeyer)
| Assignee | ||
Updated•16 years ago
|
Attachment #425208 -
Flags: review?(cpeyer) → review+
Comment 5•16 years ago
|
||
Comment on attachment 425208 [details] [diff] [review]
Update size report
Pushed size report patch:
tr-argo: 3660:b4f4ca2a33e8
tr: 3744:b4f4ca2a33e8
Updated•16 years ago
|
Status: REOPENED → RESOLVED
Closed: 16 years ago → 16 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•