Skip to content

optee_rust_examples_ext: use 'rust install toolchain' #818

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

Merged
merged 1 commit into from
Mar 4, 2025

Conversation

jforissier
Copy link
Contributor

@jforissier jforissier commented Mar 4, 2025

Invoke 'rust install toolchain' instead of relying on a side effect of 'cargo --version'. It looks like the side effect doesn't exist anymore. From optee_os CI:

2025-03-04T10:15:19.2947623Z Configuring OP-TEE rust examples
2025-03-04T10:15:19.3387454Z error: toolchain 'nightly-2023-12-18-x86_64-unknown-linux-gnu' is not installed
2025-03-04T10:15:19.3389280Z help: run rustup toolchain install nightly-2023-12-18-x86_64-unknown-linux-gnu to install it
2025-03-04T10:15:19.3403890Z make[2]: *** [package/pkg-generic.mk:273: /__w/optee_test/optee_repo_qemu_v8/out-br/build/optee_rust_examples_ext-1.0/.stamp_configured] Error 1
2025-03-04T10:15:19.3405144Z make[2]: *** Waiting for unfinished jobs....

@jforissier
Copy link
Contributor Author

Ping @b49020

@b49020
Copy link
Contributor

b49020 commented Mar 4, 2025

Sounds like a better alternative as per rust-lang/rustup#2686 too.

FWIW:

Reviewed-by: Sumit Garg <[email protected]>

@jforissier
Copy link
Contributor Author

Thanks Sumit!

@jforissier
Copy link
Contributor Author

FWIW:

Reviewed-by: Sumit Garg <[email protected]>

Should I use your kernel.org address instead perhaps...?

Invoke 'rust install toolchain' instead of relying on a side effect of
'cargo --version'. It looks like the side effect doesn't exist anymore.
From optee_os CI:

 2025-03-04T10:15:19.2947623Z Configuring OP-TEE rust examples
 2025-03-04T10:15:19.3387454Z error: toolchain 'nightly-2023-12-18-x86_64-unknown-linux-gnu' is not installed
 2025-03-04T10:15:19.3389280Z help: run `rustup toolchain install nightly-2023-12-18-x86_64-unknown-linux-gnu` to install it
 2025-03-04T10:15:19.3403890Z make[2]: *** [package/pkg-generic.mk:273: /__w/optee_test/optee_repo_qemu_v8/out-br/build/optee_rust_examples_ext-1.0/.stamp_configured] Error 1
 2025-03-04T10:15:19.3405144Z make[2]: *** Waiting for unfinished jobs....

Signed-off-by: Jerome Forissier <[email protected]>
Reviewed-by: Sumit Garg <[email protected]>
@b49020
Copy link
Contributor

b49020 commented Mar 4, 2025

Should I use your kernel.org address instead perhaps...?

My Linaro email is still active for few days, so I am using that for reviews as of now.

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