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

mbedtls: Add purl/cpe information #78599

Merged
merged 1 commit into from
Oct 8, 2024
Merged

mbedtls: Add purl/cpe information #78599

merged 1 commit into from
Oct 8, 2024

Conversation

ceolin
Copy link
Member

@ceolin ceolin commented Sep 17, 2024

Get revision with purl/cpe support.

@zephyrbot zephyrbot added the size: XS A PR changing only a single line of code label Sep 17, 2024
@zephyrbot
Copy link
Collaborator

zephyrbot commented Sep 17, 2024 •
edited
Loading

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
mbedtls zephyrproject-rtos/mbedtls@fb36f3f zephyrproject-rtos/mbedtls@eb55f47 (zephyr) zephyrproject-rtos/mbedtls@fb36f3fe..eb55f473

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@zephyrbot zephyrbot added manifest manifest-mbedtls DNM This PR should not be merged (Do Not Merge) labels Sep 17, 2024
kartben
kartben previously approved these changes Sep 17, 2024
View reviewed changes
Copy link
Collaborator

@kartben kartben left a comment

Choose a reason for hiding this comment

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

Aww nice. I was certain I had already sent a PR for this but I guess not :)

tomi-font
tomi-font previously approved these changes Sep 18, 2024
View reviewed changes
d3zd3z
d3zd3z requested changes Oct 4, 2024
View reviewed changes
Copy link
Collaborator

@d3zd3z d3zd3z left a comment

Choose a reason for hiding this comment

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

61 has merged, does this need to be updated, or the PR discarded if something else brought it in.

@ceolin
mbedtls: Add purl/cpe information
3318d7c 
Get revision with purl/cpe support.

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>
@ceolin ceolin dismissed stale reviews from tomi-font and kartben via 3318d7c October 4, 2024 21:53
@ceolin
Copy link
Member Author

ceolin commented Oct 4, 2024

@d3zd3z @kartben @tomi-font I have updated this pr now with the proper hash.

@zephyrbot zephyrbot removed the DNM This PR should not be merged (Do Not Merge) label Oct 4, 2024
@nashif nashif merged commit c60d060 into zephyrproject-rtos:main Oct 8, 2024
25 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomi-font tomi-font tomi-font approved these changes

@kartben kartben kartben approved these changes

@d3zd3z d3zd3z d3zd3z approved these changes

Assignees
No one assigned
Labels
manifest manifest-mbedtls size: XS A PR changing only a single line of code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@ceolin @zephyrbot @d3zd3z @kartben @tomi-font @nashif