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

Add missing include of inttypes.h in rf-ble-cmd.c #2889

Merged
merged 1 commit into from
May 3, 2024

Conversation

arurke
Copy link
Member

@arurke arurke commented Apr 24, 2024

As in title.

@arurke arurke changed the title Add missing inttypes.h in rf-ble-cmd.c Add missing include of inttypes.h in rf-ble-cmd.c Apr 24, 2024
@arurke
Copy link
Member Author

arurke commented Apr 24, 2024

@pjonsson Are you familiar with the failures in all the mac os jobs? A quick google did not offer me much.

@pjonsson
Copy link
Contributor

@arurke it seems the Github runners got a new macOS version, and that switched them from Intel to M1 silicon. I'm just heading into a meeting, can you please make a PR here that switches from macos-latest to the older version used by the Cooja repository? It's not just the matrix that needs to be updated, there are also checks further down that needs to be updated, brew for example is only executed on macOS.

@nfi
Copy link
Member

nfi commented Apr 25, 2024

I made a PR to address this issue: #2890

@arurke arurke force-pushed the aru/add-missing-include-in-rf-ble branch from 10fd278 to 6b63407 Compare May 1, 2024 21:19
@arurke arurke force-pushed the aru/add-missing-include-in-rf-ble branch from 6b63407 to 5df7c4d Compare May 3, 2024 14:00
@nfi nfi merged commit 99f61f5 into contiki-ng:develop May 3, 2024
24 checks passed
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.

None yet

3 participants