Skip to content

Conversation

gilles-peskine-arm
Copy link
Contributor

@gilles-peskine-arm gilles-peskine-arm commented Jul 31, 2025

A few improvements to all.sh and make_generated_files.py, mostly relevant for 3.6:

  • Have make_generated_files.py work in 3.6 as well. It's trivial and more convenient. Needs 3.6 only: make list_generated_files mbedtls#10339 for make_generated_files.py --list in 3.6.
  • Prepare for make list_generated_files to work even when called from all.sh with the make ”quiet wrapper“ active.
  • Fix make_generated_files.py --root DIR not working.
  • In all.sh cleanup, do remove $OUT_OF_SOURCE_DIR in mbedtls too (3.6 and development). We use it so we should clean it.

PR checklist

  • TF-PSA-Crypto PR not required because: just framework improvements
  • development PR not required because: just framework improvements
  • 3.6 PR not required because: just framework improvements

Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
The `--root` option was mostly ineffective since you could only use if the
current directory was some other mbedtls or TF-PSA-Crypto root.

Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
Just call `make generated_files`.

`--list` won't work yet, because 3.6 currently lacks a way to list the
generated files. We'll need to add support for `make list_generated_files`.

Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
@gilles-peskine-arm gilles-peskine-arm added enhancement New feature or request needs-ci Needs to pass CI tests priority-medium Medium priority - this can be reviewed as time permits size-xs Estimated task size: extra small (a few hours at most) labels Jul 31, 2025
Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
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-ci Needs to pass CI tests priority-medium Medium priority - this can be reviewed as time permits size-xs Estimated task size: extra small (a few hours at most)
Projects
Status: In Development
Development

Successfully merging this pull request may close these issues.

1 participant