Skip to content

chore(sdk): Move the SDK to its develop at ff54571 - #64

Merged
jbenedictocenteno merged 1 commit into
developfrom
chore/sdk-host-suite-fix
Nov 13, 2026
Merged

jbenedictocenteno merged 1 commit into
developfrom
chore/sdk-host-suite-fix

Conversation

@jbenedictocenteno

Copy link
Copy Markdown
Contributor

Description of Change

The sdk submodule and the four pins of components/kgui/idf_component.yml move from d62a89b to ff54571.

The SDK's main host suite did not build under CI's GCC at d62a89b: a case threw away fread()'s result (kodediy/kode-sdk#116). This repo's host job builds the SDK it pins, so it failed the same way on #63. ff54571 is that commit with the case fixed. No code that ships in kodeOS differs between the two.

Test Scenarios

  • The SDK's host suites pass in its CI at ff54571: 5490 cases across 8 suites.
  • This tree at d62a89b built locally on v6.1 with no warnings, and passed the size, RAM, layer and MISRA checks, the tool tests and the host suites; two DVT boards run that code.

Related links

The SDK's main host suite did not build under CI's GCC at d62a89b: a
case threw away fread()'s result. ff54571 is that commit with the case
fixed, so the submodule and kgui's four pins name it. No code that
ships in kodeOS differs between the two.
@github-actions

Copy link
Copy Markdown

kodeOS build

Check Result
Image size kodeos.bin 3865712 of 4194304 bytes, 328592 left (7%)
Internal RAM kodeos.elf 35 statics of 2048 bytes or more in internal RAM, all listed
Starts starts: 23 Active Objects and 6 door set-ups, each beside what it starts
MISRA MISRA: 1835 findings in 187 files, none past the baseline
::warning::kodeos.bin is within 10% of its ceiling (328592 bytes, 7% left of 4194304)

@jbenedictocenteno
jbenedictocenteno merged commit a0546b2 into develop Nov 13, 2026
10 checks passed
@jbenedictocenteno
jbenedictocenteno deleted the chore/sdk-host-suite-fix branch November 13, 2026 10:44
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.

1 participant