You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@teaclave.apache.org by GitBox <gi...@apache.org> on 2021/09/23 05:53:23 UTC
[GitHub] [incubator-teaclave-trustzone-sdk] DemesneGH opened a new pull request #45: Add expect script for OP-TEE CI
DemesneGH opened a new pull request #45:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/45
Add `ci/qemu-check.exp` which invoked in OP-TEE/build/qemu_v8.mk
CI for Rust applications in OP-TEE can be triggered by:
```
$ cd OPTEE_DIR/build && make CFG_TEE_CORE_LOG_LEVEL=0 check-rust
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org
[GitHub] [incubator-teaclave-trustzone-sdk] mssun commented on pull request #45: Add expect script for OP-TEE CI
Posted by GitBox <gi...@apache.org>.
mssun commented on pull request #45:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/45#issuecomment-926064558
Can you also add this in our CI? https://github.com/apache/incubator-teaclave-trustzone-sdk/blob/master/.github/workflows/ci.yml
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org
[GitHub] [incubator-teaclave-trustzone-sdk] DemesneGH commented on pull request #45: Add expect script for OP-TEE CI
Posted by GitBox <gi...@apache.org>.
DemesneGH commented on pull request #45:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/45#issuecomment-927250048
Since the new CI job clones OP-TEE upstream repositories (`optee_rust` has been pinned to a fixed version in `OP-TEE/manifest`), I think we should follow these steps before merging the CI job:
1. Merge `ci/qemu-check.exp` into `Teaclave Trustzone SDK`.
2. Merge https://github.com/DemesneGH/build/commit/31dc606401e29991dc5cee908a991bf814009f3c into `OP-TEE/build`.
3. Pin `optee_rust` to the latest version in `OPTEE/manifest`.
Then we can merge the CI job into Azure Pipelines or Github Actions.
Is this the proper way?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org
[GitHub] [incubator-teaclave-trustzone-sdk] DemesneGH commented on pull request #45: Add expect script for OP-TEE CI
Posted by GitBox <gi...@apache.org>.
DemesneGH commented on pull request #45:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/45#issuecomment-925538027
Related commit in OPTEE/build:
https://github.com/DemesneGH/build/commit/31dc606401e29991dc5cee908a991bf814009f3c
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org
[GitHub] [incubator-teaclave-trustzone-sdk] mssun commented on pull request #45: Add expect script for OP-TEE CI
Posted by GitBox <gi...@apache.org>.
mssun commented on pull request #45:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/45#issuecomment-928106823
@DemesneGH, thanks!
Ok, sure. Let's do it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org
[GitHub] [incubator-teaclave-trustzone-sdk] DemesneGH commented on pull request #45: Add expect script for OP-TEE CI
Posted by GitBox <gi...@apache.org>.
DemesneGH commented on pull request #45:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/45#issuecomment-926364854
> Can you also add this in our CI? https://github.com/apache/incubator-teaclave-trustzone-sdk/blob/master/.github/workflows/ci.yml
@mssun I have written the ci script in this branch https://github.com/DemesneGH/rust-optee-trustzone-sdk/blob/test-ci/.github/workflows/ci.yml and it worked. But in that script, I had to checkout my commits which have not been merged yet (https://github.com/DemesneGH/rust-optee-trustzone-sdk/blob/test-ci/.github/workflows/ci.yml#L38-L43).
Shall we add this to our CI now? Or add this after merging this PR and the PR in `OP-TEE/build`?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org
[GitHub] [incubator-teaclave-trustzone-sdk] mssun commented on pull request #45: Add expect script for OP-TEE CI
Posted by GitBox <gi...@apache.org>.
mssun commented on pull request #45:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/45#issuecomment-926788556
You can commit the changes of `ci.yml` here. GitHub Actions will automatically run based on the latest `ci.yml` file.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org
[GitHub] [incubator-teaclave-trustzone-sdk] mssun commented on pull request #45: Add expect script for OP-TEE CI
Posted by GitBox <gi...@apache.org>.
mssun commented on pull request #45:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/45#issuecomment-928106823
@DemesneGH, thanks!
Ok, sure. Let's do it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org
[GitHub] [incubator-teaclave-trustzone-sdk] mssun merged pull request #45: Add expect script for OP-TEE CI
Posted by GitBox <gi...@apache.org>.
mssun merged pull request #45:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/45
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org
[GitHub] [incubator-teaclave-trustzone-sdk] mssun merged pull request #45: Add expect script for OP-TEE CI
Posted by GitBox <gi...@apache.org>.
mssun merged pull request #45:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/45
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org