fix(ci): build Linux ARM releases natively - #29
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d601d8e4ea
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| push: | ||
| tags: | ||
| - 'v*' | ||
| workflow_dispatch: |
There was a problem hiding this comment.
Sanitize dispatched branch names before packaging
When this workflow is manually dispatched against a slash-delimited branch such as feature/foo, github.ref_name retains that slash, but the packaging steps interpolate it directly into the archive filename. The Unix command consequently tries to write beneath a nonexistent code-memory-feature/ directory and fails, so the newly added manual build verification cannot run on common feature-branch names; derive a filesystem-safe artifact suffix or omit the ref name for dispatch runs.
Useful? React with 👍 / 👎.
Summary
openssl-sysRoot cause
The
v0.1.0release run failed while cross-compilingaarch64-unknown-linux-gnubecausefastembed -> hf-hub -> native-tlsrequires an ARM64 OpenSSL development toolchain. Installing only the ARM GCC/G++ cross compilers does not provide it.Verification
.github/workflows/release.ymlsuccessfully as YAMLcargo test --all-features(all tests pass; one integration test intentionally ignored)cargo fmt --package code-memory -- --checkcargo clippy --all-targets --all-features -- -D warningsgit diff --checkAfter merge, the new manual dispatch can verify the complete release build matrix without creating a release or publishing the npm package.