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

Move generate scripts #114

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Harry-Ramsey
Copy link
Contributor

Description

Moves generate-script files from Mbed TLS to Mbed TLS framework.

PR checklist

Please add the numbers (or links) of the associated pull requests for consuming branches. You can omit branches where this pull request is not needed.

  • crypto PR Mbed-TLS/TF-PSA-Crypto# TODO
  • development PR Mbed-TLS/mbedtls#
  • 3.6 PR Mbed-TLS/mbedtls# TODO

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 7, 2025
@Harry-Ramsey Harry-Ramsey force-pushed the move-generate-scripts branch 2 times, most recently from ed0f5ed to 0fcf401 Compare January 8, 2025 15:17
This commit moves generate_features.pl and generate_query_config.pl into
the framework.

Signed-off-by: Harry Ramsey <[email protected]>
This commit adapts the file generate_query_config.pl to work for both
Mbed TLS 3.6 and development versions.

Signed-off-by: Harry Ramsey <[email protected]>
@Harry-Ramsey Harry-Ramsey force-pushed the move-generate-scripts branch from 0fcf401 to 079f3c7 Compare January 8, 2025 15:33
@Harry-Ramsey
Copy link
Contributor Author

Not too sure where the history went? I used to python script however it may seem that all those previous changes were merged?

@Harry-Ramsey Harry-Ramsey added 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 size-xs Estimated task size: extra small (a few hours at most) labels Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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 size-xs Estimated task size: extra small (a few hours at most)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant