Skip to content

Conversation

@petermm
Copy link
Contributor

@petermm petermm commented Sep 15, 2025

CI for release was using outdated/wrong way of building atomvmlib-pico.uf2/atomvmlib-pico2.uf2, fixed to the new ninja building.

Moved said building to a job before the various pico builds, with the necessary upload/download of artifacts.

Removed wrong 'mv atomvmlib.uf2' segment, file is non-existent on new ninja builds, and was a leftover/duplicated code.

Added combined uf2 artifacts using uf2tool.

Upload the artifacts, so they are available for testing.

Include combined artifact in release.

Link to CI run/artifacts: https://github.com/atomvm/AtomVM/actions/runs/18612666167

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).

SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

@petermm petermm marked this pull request as draft September 15, 2025 11:12
@petermm petermm force-pushed the ci-pico-fix-and-artifacts branch 3 times, most recently from 592f21c to bb407c4 Compare September 15, 2025 12:36
@petermm petermm marked this pull request as ready for review September 15, 2025 12:59
@petermm petermm force-pushed the ci-pico-fix-and-artifacts branch from bb407c4 to de48b08 Compare October 18, 2025 07:29
@petermm
Copy link
Contributor Author

petermm commented Oct 18, 2025

@petermm petermm requested a review from pguyot October 18, 2025 08:38
CI for release was using outdated way of building atomvmlib-pico.uf2/atomvmlib-pico2.uf2, fixed to the ninja building.

Moved said building to a job before, the various pico builds.

Removed wrong 'mv atomvmlib.uf2' segment, file is non-existent on new ninja builds.

Added combined uf2 artifacts using uf2tool.

Upload the artifacts, so they are available for testing.

Include combined artifact in release.

Signed-off-by: Peter M <[email protected]>
@petermm petermm force-pushed the ci-pico-fix-and-artifacts branch from de48b08 to 51a0681 Compare October 30, 2025 10: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