Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: skip a wasi test on IBMi PASE #32459

Closed
wants to merge 1 commit into from
Closed

test: skip a wasi test on IBMi PASE #32459

wants to merge 1 commit into from

Conversation

dmabupt
Copy link
Contributor

@dmabupt dmabupt commented Mar 24, 2020

The API clock_getres with attribute API CLOCK_THREAD_CPUTIME_ID
is not supported on IBM i PASE.

This is the same change as V8 --> https://github.com/nodejs/node/blob/master/deps/v8/src/base/platform/time.cc#L73

@nodejs-github-bot nodejs-github-bot added the test Issues and PRs related to the tests. label Mar 24, 2020
@cjihrig
Copy link
Contributor

cjihrig commented Mar 24, 2020

This won't do what you think it does. You need to skip the test on IBMi from here.

@dmabupt
Copy link
Contributor Author

dmabupt commented Mar 24, 2020

This won't do what you think it does. You need to skip the test on IBMi from here.

Ah... I thought that would skip all the 4 tests. So the c code will not be compiled to wasm automatically?

Since this line also failed, I may investigate a little more and edit the test-wasi.js directly.

@cjihrig
Copy link
Contributor

cjihrig commented Mar 24, 2020

So the c code will not be compiled to wasm automatically?

No, because the CI and users would need to have the WASI SDK installed to compile from .c to .wasm. But, beyond that, .wasm files should be platform independent I think.

The API clock_getres with attribute API CLOCK_THREAD_CPUTIME_ID
is not supported on IBM i PASE.
@addaleax addaleax added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Mar 29, 2020
@nodejs-github-bot
Copy link
Collaborator

@addaleax
Copy link
Member

Landed in 4898c1d

@addaleax addaleax closed this Mar 30, 2020
addaleax pushed a commit that referenced this pull request Mar 30, 2020
The API clock_getres with attribute API CLOCK_THREAD_CPUTIME_ID
is not supported on IBM i PASE.

PR-URL: #32459
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
addaleax pushed a commit that referenced this pull request Mar 30, 2020
The API clock_getres with attribute API CLOCK_THREAD_CPUTIME_ID
is not supported on IBM i PASE.

PR-URL: #32459
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@dmabupt dmabupt deleted the ibmi_wasi_clock_getres branch April 14, 2020 00:08
targos pushed a commit that referenced this pull request Apr 22, 2020
The API clock_getres with attribute API CLOCK_THREAD_CPUTIME_ID
is not supported on IBM i PASE.

PR-URL: #32459
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants