Mbed-TLS / TF-PSA-Crypto

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

Add semantic versioning #40

Closed davidhorstmann-arm closed 11 months ago

davidhorstmann-arm commented 12 months ago

Add a version number to PSA Crypto. Use CMake to configure this automatically.

Note that this is based on top of #38 to avoid conflicts in CMakeLists.txt. Only the top 3 commits actually relate to this PR.

Fixes #30

davidhorstmann-arm commented 11 months ago

This should be ready for review.

In order to make the tests work, I have had to add standard function aliases to include/psa/platform.h which I think were necessary, but I may have misunderstood the way the platform interface is supposed to work.

Let me know if I should change this to a different approach.

ronald-cron-arm commented 11 months ago

In order to make the tests work, I have had to add standard function aliases to include/psa/platform.h which I think were necessary, but I may have misunderstood the way the platform interface is supposed to work.

Yes I think this is fine.

davidhorstmann-arm commented 11 months ago

I believe I've now covered all occurrences of the version number in the project, so it should just be the top-level CMakeLists.txt that contains the version number.

davidhorstmann-arm commented 11 months ago

Apologies for the force-push, this is now rebased atop development and should be ready for re-review.

I did not change anything in the force-push except for doing the rebase.

davidhorstmann-arm commented 11 months ago

Apologies for the delay, this should be ready for re-review.