search

Mbed-TLS / TF-PSA-Crypto

Reference implementation of the PSA Cryptography API
Apache License 2.0
9 stars 7 forks source link

Adapt to new repository name #82

Closed ronald-cron-arm closed 1 year ago

ronald-cron-arm commented 1 year ago

Completeness check using: grep -r -i -E "([^-]|[^f][-]|[^t]f[_-])psa[^/]crypto([^g]|g[^r])" to avoid "tf.psa.crypto" and "psa.cryptogr" patterns. It seems reasonable to keep the "psa.crypto" patterns returned by the above search (mainly symbols shared with mbedtls repo).

davidhorstmann-arm commented 1 year ago

Thinking about it, it would be good to rename scripts/psa_crypto.py and names in it as well, since that refers to the repo not the API.

ronald-cron-arm commented 1 year ago

Thinking about it, it would be good to rename scripts/psa_crypto.py and names in it as well, since that refers to the repo not the API.

Well, we can think about it as the script to copy the PSA crypto implementation into TF-PSA-Crypto (aligned with the script help) thus I think it is okay to keep it as it is. But I can change it if you prefer.

davidhorstmann-arm commented 1 year ago

Well, we can think about it as the script to copy the PSA crypto implementation into TF-PSA-Crypto (aligned with the script help) thus I think it is okay to keep it as it is. But I can change it if you prefer.

Okay, sounds fine to me.

davidhorstmann-arm commented 1 year ago

Checked using the silver searcher with the following:

ag '(?<!tf-)psa-crypto'
ag '(?<!tf_)psa_crypto'
ag '(?<!tf)psacrypto'
davidhorstmann-arm commented 1 year ago

Ah I need Mbed-TLS/mbedtls#8327, let me retest with that.