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.

Sign up for GitHub

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

Jump to bottom

[tf-psa-crypto] Add test_tf_psa_crypto_cmake_shared to components-build-system.sh #181

Open
wants to merge 7 commits into
base: development
Choose a base branch
from
Open

[tf-psa-crypto] Add test_tf_psa_crypto_cmake_shared to components-build-system.sh #181

wants to merge 7 commits into from

Conversation

valeriosetti
Copy link
Contributor

@valeriosetti valeriosetti commented Feb 25, 2025 •
edited
Loading

Description

Part of #138

PR checklist

@valeriosetti
programs: re-add dlopen
c73538f 
dlopen was unintentionally removed in
d3b897d. This commit just adds it back
with:

    git checkout d3b897d -- programs/test/dlopen.c

but it does not modify its content. This will be done in follow-up
commits.

Signed-off-by: Valerio Setti <vsetti@baylibre.com>
@valeriosetti
programs: fix dlopen to build in tf-psa-crypto
0ee36ea 
This commit also adds support for building dlopen in CMakeList.txt.

Signed-off-by: Valerio Setti <vsetti@baylibre.com>
@valeriosetti
programs: fix cmake source file
790ad5d 
Program files do not have a 100% common pattern to be built, so intead
of a for-loop, we create a common function that sets up what can be
shared and then each program is built on its own.

Signed-off-by: Valerio Setti <vsetti@baylibre.com>
@valeriosetti
programs: rename dlopen target for tf-sa-crypto repo
304a386 
This avoid target name clashing with the dlopen program in the main
repository.

Signed-off-by: Valerio Setti <vsetti@baylibre.com>
@valeriosetti
test: add component to test shared library build and dlopen
68df142 
Signed-off-by: Valerio Setti <vsetti@baylibre.com>
@valeriosetti
programs: do not specify dynamic library path in dlopen
753ede8 
The path of the dynamic library should be set using LD_LIBRARY_PATH
environment variable.

Signed-off-by: Valerio Setti <vsetti@baylibre.com>
@valeriosetti
framework: update reference
ea26dea 
Signed-off-by: Valerio Setti <vsetti@baylibre.com>
@valeriosetti valeriosetti added size-s Estimated task size: small (~2d) 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 priority-high High priority - will be reviewed soon needs-ci Needs to pass CI tests labels Feb 25, 2025
@valeriosetti valeriosetti self-assigned this Feb 25, 2025
This was referenced Feb 25, 2025
Open
Open
@valeriosetti valeriosetti removed the needs-ci Needs to pass CI tests label Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Harry-Ramsey Harry-Ramsey Harry-Ramsey approved these changes

@ronald-cron-arm ronald-cron-arm Awaiting requested review from ronald-cron-arm

At least 2 approving reviews are required to merge this pull request.

Assignees

@valeriosetti valeriosetti

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 priority-high High priority - will be reviewed soon size-s Estimated task size: small (~2d)
Projects
Mbed TLS Epics
Status: TF-PSA-Crypto all.sh 2 - misc
Roadmap pull requests (new board)
Status: In Development
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@valeriosetti @Harry-Ramsey