Skip to content

Enable IRT tests - #76

Open
slipher wants to merge 3 commits into
masterfrom
slipher/irt-tests
Open

slipher wants to merge 3 commits into
masterfrom
slipher/irt-tests

Conversation

@slipher

@slipher slipher commented Oct 14, 2026

Copy link
Copy Markdown
Member

See #64

@slipher
slipher force-pushed the slipher/irt-tests branch 2 times, most recently from d2884be to 908869f Compare October 14, 2026 11:29
@slipher
slipher marked this pull request as ready for review October 14, 2026 11:52
illwieckz
illwieckz previously approved these changes Oct 14, 2026

@illwieckz illwieckz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks.

LGTM.

@slipher

slipher commented Oct 16, 2026

Copy link
Copy Markdown
Member Author

Somehow this is not working. None of the IRT tests are actually building or running in the CI.

@slipher
slipher dismissed illwieckz’s stale review October 16, 2026 14:43

it's not working :(

It was flaky because the IRT thread exit hook really hooks the thread
exit syscall rather than pthread_exit. So the hook can surprisingly run
after the thread is already joinable.
Also document them a bit in the README.

Fixes #64.
@slipher

slipher commented Oct 27, 2026

Copy link
Copy Markdown
Member Author

Should work now (i.e. actually run IRT tests on the CI). I had enabled the nacl_irt_test mode in the build phase, but not the test phase.

Also added another commit to find nacl-objcopy in the SDK.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants