mbedtls icon indicating copy to clipboard operation
mbedtls copied to clipboard

Adapt tests/scripts/check_names.py to work for psa-crypto

Open gabor-mezei-arm opened this issue 1 year ago • 3 comments

Description

Resolve Mbed-TLS/TF-PSA-Crypto#52

PR checklist

Please tick as appropriate and edit the reasons (e.g.: "backport: not needed because this is a new feature")

  • [x] changelog not required, test framework
  • [ ] 3.6 backport ???
  • [x] 2.28 backport not required, new feature
  • [x] tests not required, already has

gabor-mezei-arm avatar Dec 21 '23 18:12 gabor-mezei-arm

@gabor-mezei-arm Do I need to do anything special to get the check_names.py script to run in the TF-PSA-Crypto repository?

I have created a fresh clone of TF-PSA-Crypto based on the main branch, copied the modified check_names.py and build_tree.py scripts into it and tried to run the script but I am getting errors from the compilation stage.

Have I missed a key step to getting this to run, or should this just work?

tom-daubney-arm avatar Jan 05 '24 16:01 tom-daubney-arm

Do I need to do anything special to get the check_names.py script to run in the TF-PSA-Crypto repository?

Currently the check_names.py is not working with the TF-PSA-Crypto repo because:

  • the development branch in the TF-PSA-Crypto repo is not building
  • it is using the config.py which is not ported yet

gabor-mezei-arm avatar Jan 05 '24 16:01 gabor-mezei-arm

Do I need to do anything special to get the check_names.py script to run in the TF-PSA-Crypto repository?

Currently the check_names.py is not working with the TF-PSA-Crypto repo because:

* the `development` branch in the `TF-PSA-Crypto` repo is not building

* it is using the `config.py` which is not ported yet

Ok thanks. I wasn't using the development branch however the errors I was getting were from config.py being missing so that all makes sense.

tom-daubney-arm avatar Jan 05 '24 19:01 tom-daubney-arm