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

[Backport v4.0-branch] manifest: update percepio #83290

Merged
merged 1 commit into from
Dec 27, 2024
Merged

[Backport v4.0-branch] manifest: update percepio #83290

merged 1 commit into from
Dec 27, 2024

Conversation

zephyrbot
Copy link
Collaborator

@zephyrbot zephyrbot commented Dec 20, 2024 •
edited by eriktamlin
Loading

Backport 91b2156 from #83180.

Fixes #83182

@eriktamlin @github-actions
manifest: update percepio
c457fe8 
Update the percepio module to use TraceRecorder v4.10.2

Signed-off-by: Erik Tamlin <[email protected]>
(cherry picked from commit 91b2156)
@zephyrbot zephyrbot added Backport Backport PR and backport failure issues size: XS A PR changing only a single line of code labels Dec 20, 2024
@zephyrbot
Copy link
Collaborator Author

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

Name Old Revision New Revision Diff
percepio zephyrproject-rtos/percepio@b68d179 zephyrproject-rtos/percepio@0d44033 (zephyr) zephyrproject-rtos/[email protected]

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

@eriktamlin eriktamlin linked an issue Dec 23, 2024 that may be closed by this pull request
v4.0-branch: manifest: update percepio #83182
Open
@dkalowsk
Copy link
Contributor

@kartben I'm hoping for a little clarification here on when we update releases and versions of 3rd party software. When @mmahadevan108 tried to merge a CMSIS version update (that was scheduled for the original release but missed the deadline), there was push back saying (if I recall correctly) that we don't change versions in releases except for LTS. This is updating the percepio to a new version, does it not fall into the same category?

@kartben
Copy link
Collaborator

kartben commented Dec 27, 2024 •
edited
Loading

@kartben I'm hoping for a little clarification here on when we update releases and versions of 3rd party software. When @mmahadevan108 tried to merge a CMSIS version update (that was scheduled for the original release but missed the deadline), there was push back saying (if I recall correctly) that we don't change versions in releases except for LTS. This is updating the percepio to a new version, does it not fall into the same category?

@dkalowsk
This one is effectively fixing a bug, according to the linked issue, which I don't think the CMSIS stuff was?

@dkalowsk
Copy link
Contributor

@dkalowsk This one is effectively fixing a bug, according to the linked issue, which I don't think the CMSIS stuff was?

Alright so the break down is enhancement vs bug fix. Thanks for confirming this!

@dkalowsk dkalowsk added this to the v4.0.1 milestone Dec 27, 2024
@dkalowsk dkalowsk merged commit 3f4037b into v4.0-branch Dec 27, 2024
33 of 34 checks passed
@dkalowsk dkalowsk deleted the backport-83180-to-v4.0-branch branch December 27, 2024 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dkalowsk dkalowsk dkalowsk approved these changes

@ycsin ycsin ycsin approved these changes

@kartben kartben kartben approved these changes

@nashif nashif Awaiting requested review from nashif

@henrikbrixandersen henrikbrixandersen Awaiting requested review from henrikbrixandersen

Assignees
No one assigned
Labels
Backport Backport PR and backport failure issues manifest manifest-percepio size: XS A PR changing only a single line of code
Projects
Backports
Status: Done
Milestone
v4.0.1
Development

Successfully merging this pull request may close these issues.

v4.0-branch: manifest: update percepio
5 participants
@zephyrbot @dkalowsk @kartben @ycsin @eriktamlin