Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Move pkgconfig.sh to the framework #116

Merged
merged 6 commits into from
Jan 14, 2025
Merged

Move pkgconfig.sh to the framework #116

merged 6 commits into from
Jan 14, 2025

Conversation

valeriosetti
Copy link
Contributor

@valeriosetti valeriosetti commented Jan 9, 2025 •
edited
Loading

Description

Resolves #101

PR checklist

Please add the numbers (or links) of the associated pull requests for consuming branches. You can omit branches where this pull request is not needed.

Note to self: the file was moved with:

../mbedtls-docs/tools/bin/mbedtls-move-to-framework \
--from . --to ../mbedtls-framework \
--src-branch issue101-development \
--dst-branch issue101-framework \
--path tests/scripts/pkgconfig.sh:scripts/pkgconfig.sh

valeriosetti and others added 6 commits January 8, 2025 09:51
@valeriosetti
Move files out of Mbed TLS
cb333f2 
The following files are moved to the framework repo (deleted here):

scripts/assemble_changelog.py
tests/scripts/check-doxy-blocks.pl
tests/scripts/check-python-files.sh
tests/scripts/doxygen.sh
scripts/apidoc_full.sh
tests/scripts/recursion.pl

Signed-off-by: Valerio Setti <valerio.setti@nordicsemi.no>
@valeriosetti
scripts: fix paths for files moved to framework
976b53f 
Signed-off-by: Valerio Setti <valerio.setti@nordicsemi.no>
@valeriosetti
framework: updated reference
5300725 
Signed-off-by: Valerio Setti <valerio.setti@nordicsemi.no>
@ronald-cron-arm
Merge pull request #9825 from valeriosetti/issue69-main
272ba23 
Move "easy" basic checks scripts to the framework
@valeriosetti
Move files into the framework
98fe7ee 
The following files are added (imported) from the main Mbed TLS repo:

scripts/pkgconfig.sh

Signed-off-by: Valerio Setti <valerio.setti@nordicsemi.no>
@valeriosetti
Merge branch 'tmp-branch-move-files-to-framework' into issue101-frame…
9753be6 
…work
@valeriosetti valeriosetti self-assigned this Jan 9, 2025
@valeriosetti valeriosetti added 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-xs Estimated task size: extra small (a few hours at most) labels Jan 9, 2025
This was referenced Jan 9, 2025
Merged
Merged
ronald-cron-arm
ronald-cron-arm approved these changes Jan 10, 2025
View reviewed changes
Copy link
Contributor

@ronald-cron-arm ronald-cron-arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Harry-Ramsey
Harry-Ramsey approved these changes Jan 13, 2025
View reviewed changes
Copy link
Contributor

@Harry-Ramsey Harry-Ramsey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ronald-cron-arm ronald-cron-arm merged commit 57ee691 into Mbed-TLS:main Jan 14, 2025
1 of 2 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@ronald-cron-arm ronald-cron-arm ronald-cron-arm approved these changes

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

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-xs Estimated task size: extra small (a few hours at most)
Projects
Roadmap pull requests (new board)
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Move pkgconfig.sh to the framework
3 participants
@valeriosetti @ronald-cron-arm @Harry-Ramsey