Remove avm selftest compile steps now that selftest is enabled for all configs.

VERIFIED FIXED

Status

defect
VERIFIED FIXED
10 years ago
10 years ago

People

(Reporter: cpeyer, Assigned: cpeyer)

Tracking

Details

Attachments

(2 attachments)

Assignee

Description

10 years ago
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

10 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

10 years ago
Attachment #425084 - Flags: review?(brbaker) → review+

Comment 2

10 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

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

Comment 3

10 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

10 years ago
Update the size report scripts to disable the selftests from being compiled
Attachment #425208 - Flags: review?(cpeyer)
Assignee

Updated

10 years ago
Attachment #425208 - Flags: review?(cpeyer) → review+

Comment 5

10 years ago
Comment on attachment 425208 [details] [diff] [review]
Update size report

Pushed size report patch:
tr-argo: 3660:b4f4ca2a33e8
tr: 3744:b4f4ca2a33e8

Updated

10 years ago
Status: REOPENED → RESOLVED
Closed: 10 years ago10 years ago
Resolution: --- → FIXED

Updated

10 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.