Skip to content

09-coap: do not expect a specific board name#321

Merged
miri64 merged 1 commit intoRIOT-OS:masterfrom
mguetschow:native64-rename
Apr 7, 2025
Merged

09-coap: do not expect a specific board name#321
miri64 merged 1 commit intoRIOT-OS:masterfrom
mguetschow:native64-rename

Conversation

@mguetschow
Copy link
Copy Markdown
Contributor

follow-up after RIOT-OS/RIOT#21242 where native was changed to be an alias for native32/64 depending on the host architecture

@mguetschow mguetschow requested a review from miri64 April 7, 2025 08:56
@mguetschow
Copy link
Copy Markdown
Contributor Author

A more elaborate fix would be to change all release tests explicitly to native64 (which now happened implicitly).

miri64
miri64 previously approved these changes Apr 7, 2025
@miri64 miri64 dismissed their stale review April 7, 2025 09:04

Let's do a testrun first

@miri64
Copy link
Copy Markdown
Member

miri64 commented Apr 7, 2025

@mguetschow mguetschow changed the title 09-coap: adapt expected output to native64 09-coap: do not expect a specific board name Apr 7, 2025
@miri64
Copy link
Copy Markdown
Member

miri64 commented Apr 7, 2025

triggered as a follow-up after RIOT-OS/RIOT#21242 where native was changed to be an alias for native32/64 depending on the host architecture

Co-authored-by: Martine Lenders <martine.lenders@tu-dresden.de>
@miri64
Copy link
Copy Markdown
Member

miri64 commented Apr 7, 2025

https://github.com/RIOT-OS/RIOT/actions/runs/14305911814 (hopefully the last link ^^)

Copy link
Copy Markdown
Member

@miri64 miri64 left a comment

Choose a reason for hiding this comment

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

All native tasks that are supposed to pass, pass again.

@miri64 miri64 merged commit 5441007 into RIOT-OS:master Apr 7, 2025
3 checks passed
@mguetschow mguetschow deleted the native64-rename branch April 7, 2025 12:59
@mguetschow
Copy link
Copy Markdown
Contributor Author

Thanks!

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