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.

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

thread: adapt to API change of thread_measure_stack_free() #96

Merged
merged 1 commit into from
May 31, 2024

Conversation

maribu
Copy link
Member

@maribu maribu commented May 31, 2024

This changes the API as needed for RIOT-OS/RIOT#18942

@maribu maribu force-pushed the api-change/thread_measure_stack_free branch 2 times, most recently from 42c8cd0 to 69d709e Compare May 31, 2024 12:29
maribu added a commit to maribu/RIOT that referenced this pull request May 31, 2024
@chrysn
Copy link
Member

chrysn commented May 31, 2024

While updating in a single lockstep would work (the RIOT PR would point to this branch, then this branch can be merged once that hit RIOT master, then a follow-up could revert the change that selects a branch in to the override), I'll try whether this can be done in a somewhat more dynamic manner, allowing riot-wrappers to work both with RIOT 2024.04 and with the upcoming master.

@maribu maribu force-pushed the api-change/thread_measure_stack_free branch 3 times, most recently from 54ccb0d to e1d88b9 Compare May 31, 2024 14:30
maribu added a commit to maribu/RIOT that referenced this pull request May 31, 2024
@maribu maribu force-pushed the api-change/thread_measure_stack_free branch from e1d88b9 to c1cbf0c Compare May 31, 2024 14:31
@maribu maribu force-pushed the api-change/thread_measure_stack_free branch from c1cbf0c to 7f01844 Compare May 31, 2024 14:34
maribu added a commit to maribu/RIOT that referenced this pull request May 31, 2024
@maribu maribu merged commit 7f01844 into RIOT-OS:main May 31, 2024
46 of 59 checks passed
@maribu maribu deleted the api-change/thread_measure_stack_free branch May 31, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants