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

Add components configuration platform to TF-PSA-Crypto #150

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

Harry-Ramsey
Copy link
Contributor

@Harry-Ramsey Harry-Ramsey commented Jan 13, 2025

This commit adds multiple configurations to TF-PSA-Crypto similar to those found in Mbed TLS. Closes #68.

PR checklist

Please remove the segment/s on either side of the | symbol as appropriate, and add any relevant link/s to the end of the line.
If the provided content is part of the present PR remove the # symbol.

  • changelog not required because: testing enhancement.
  • framework PR provided Mbed-TLS/mbedtls-framework not required.
  • mbedtls PR not required.
  • tests provided.

Notes for the submitter

Please refer to the contributing guidelines, especially the
checklist for PR contributors.

Help make review efficient:

  • Multiple simple commits
    • please structure your PR into a series of small commits, each of which does one thing
  • Avoid force-push
    • please do not force-push to update your PR - just add new commit(s)
  • See our Guidelines for Contributors for more details about the review process.

@Harry-Ramsey Harry-Ramsey self-assigned this Jan 13, 2025
@Harry-Ramsey Harry-Ramsey force-pushed the components-configuration-platform branch from b1437e7 to e845cc3 Compare January 13, 2025 11:13
@Harry-Ramsey Harry-Ramsey added enhancement New feature or request size-s Estimated task size: small (~2d) needs-review Every commit must be reviewed by at least two team members needs-reviewer This PR needs someone to pick it up for review and removed needs-work labels Jan 13, 2025
@Harry-Ramsey Harry-Ramsey force-pushed the components-configuration-platform branch 2 times, most recently from c935433 to 5029b5a Compare February 4, 2025 10:28
tests/scripts/components-configuration-platform.sh Outdated Show resolved Hide resolved
tests/scripts/components-configuration-platform.sh Outdated Show resolved Hide resolved
tests/scripts/components-configuration-platform.sh Outdated Show resolved Hide resolved
tests/scripts/components-configuration-platform.sh Outdated Show resolved Hide resolved
tests/scripts/components-configuration-platform.sh Outdated Show resolved Hide resolved
tests/scripts/components-configuration-platform.sh Outdated Show resolved Hide resolved
This commit adds multiple configurations to TF-PSA-Crypto similar to
those found in Mbed TLS.

Signed-off-by: Harry Ramsey <[email protected]>
@Harry-Ramsey Harry-Ramsey force-pushed the components-configuration-platform branch from 5029b5a to 73429d4 Compare February 5, 2025 13:55
@ronald-cron-arm ronald-cron-arm self-requested a review February 6, 2025 14:08
Copy link
Contributor

@ronald-cron-arm ronald-cron-arm left a comment

Choose a reason for hiding this comment

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

LGTM

@valeriosetti valeriosetti self-requested a review February 7, 2025 14:39
@ronald-cron-arm ronald-cron-arm removed the needs-reviewer This PR needs someone to pick it up for review label Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs-review Every commit must be reviewed by at least two team members size-s Estimated task size: small (~2d)
Projects
Status: TF-PSA-Crypto all.sh 2 - misc
Development

Successfully merging this pull request may close these issues.

Add components-configuration-platform.sh
3 participants