Skip to content

Mynewt CI update#2712

Open
sjanc wants to merge 2 commits intomcu-tools:mainfrom
sjanc:mynewt_ci_update
Open

Mynewt CI update#2712
sjanc wants to merge 2 commits intomcu-tools:mainfrom
sjanc:mynewt_ci_update

Conversation

@sjanc
Copy link
Copy Markdown
Contributor

@sjanc sjanc commented Apr 24, 2026

No description provided.

@sjanc
Copy link
Copy Markdown
Contributor Author

sjanc commented Apr 27, 2026

CI failure doesn't seem to be related to this change

@sjanc sjanc force-pushed the mynewt_ci_update branch 2 times, most recently from c964eb9 to e636797 Compare April 27, 2026 12:57
@d3zd3z
Copy link
Copy Markdown
Member

d3zd3z commented Apr 27, 2026

The CI failures aren't related.

Some suggested commit text, although I'm not completely sure the reason for the 48K change, but it'd be good to capture:

  Commit 1 — b78a15aa ci: mynewt: Update actions

  ci: mynewt: modernize toolchain and inline workflow

  Replace the hand-rolled ARM toolchain install (wget of a 2017
  GCC 7 release) with carlosperate/arm-none-eabi-gcc-action@v1
  pinned to 15.2.Rel1, and install newt via "go install" instead
  of cloning and building newt from source. Both removals shed
  ~110 lines of bootstrap shell that only existed to recreate
  what the upstream actions now provide.

  Inline the install and run logic into the workflow itself and
  delete the ci/mynewt_install.sh and ci/mynewt_run.sh wrappers.

  After "newt upgrade", drop the PR checkout into repos/mcuboot
  via "git clone ..", so the build exercises the mcuboot revision
  under review rather than whatever upstream tag newt would
  otherwise resolve. Add mcuboot to project.repositories.allowed
  in ci/mynewt_project.yml so newt accepts the local repo.

  The redundant "Signed commit check" step is dropped here; DCO
  is already enforced by the sim, fih_tests, and espressif
  workflows.

  Signed-off-by: Szymon Janc <...>

  ---
  Commit 2 — e6367972 ci: mynewt: Fix RSA targets build

  ci: mynewt: enlarge bootloader slot for rsa_rsaoaep

  The rsa_rsaoaep target no longer fits in the default BSP
  bootloader flash region after the toolchain bump to GCC 15.2,
  which produces a larger image than the previous GCC 7 build.

  Set BSP_BOOTLOADER_48K=1 in the target's syscfg.yml so the
  BSP reserves a 48K bootloader slot and the build links cleanly
  again.

  Signed-off-by: Szymon Janc <...>

sjanc added 2 commits April 28, 2026 10:24
Replace the hand-rolled ARM toolchain install (wget of a 2017
GCC 7 release) with carlosperate/arm-none-eabi-gcc-action@v1
pinned to 15.2.Rel1, and install newt via "go install" instead
of cloning and building newt from source. Both removals shed
~110 lines of bootstrap shell that only existed to recreate
what the upstream actions now provide.

Inline the install and run logic into the workflow itself and
delete the ci/mynewt_install.sh and ci/mynewt_run.sh wrappers.

After "newt upgrade", drop the PR checkout into repos/mcuboot
via "git clone ..", so the build exercises the mcuboot revision
under review rather than whatever upstream tag newt would
otherwise resolve. Add mcuboot to project.repositories.allowed
in ci/mynewt_project.yml so newt accepts the local repo.

The redundant "Signed commit check" step is dropped here; DCO
is already enforced by the sim, fih_tests, and espressif
workflows.

Signed-off-by: Szymon Janc <szymon.janc@codecoup.pl>
rsa_rsaoaep and rsa_rsaoaep_bootstrap targets no longer fits in
the default BSP bootloader flash region after update to mbedtls 3.6.5
which produces a larger image than the previous mbedtls 2.28.9 build.

Set BSP_BOOTLOADER_48K=1 in the basic target's syscfg.yml so the
BSP reserves a 48K bootloader slot and the build links cleanly again.
Basic target is inherited by all other targets.

Signed-off-by: Szymon Janc <szymon.janc@codecoup.pl>
@sjanc sjanc force-pushed the mynewt_ci_update branch from e636797 to 2c30dbb Compare April 28, 2026 08:34
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