Correctly initialize and update git submodules for IDF in autobuild.sh #124

2026-01-26T12:07:12+08:00

gris-gris commented

2026-01-26 12:07:12 +08:00

(Migrated from https://github.com/polhenarejos/pico-hsm)

Fixing autobuild script so ESP32 binaries now built correctly

Summary

Describe in plain language what this PR does and why.

What problem does it solve?
Autobuild in github actions (nightly) not building ESP32 binaries correctly due to uninitialized submodules
Is it a bug fix, a new feature, a cleanup/refactor…?
CI Bug fix

Licensing confirmation (required)

By checking the box below, you confirm ALL of the following:

You are the author of this contribution, or you have the right to contribute it.
You have read CONTRIBUTING.md.
You agree that this contribution may be merged, used, modified, and redistributed:
- under the AGPLv3 Community Edition, and
- under any proprietary / commercial / Enterprise editions of this project,
  now or in the future.
You understand that submitting this PR does not create any support obligation,
SLA, or guarantee of merge.

I confirm the above licensing terms:

Yes, I agree

Anything else?

Optional: mention known limitations, follow-ups, or if this is related to an existing Issue.

Fixing autobuild script so ESP32 binaries now built correctly ## Summary Describe in plain language what this PR does and why. - What problem does it solve? **Autobuild in github actions (nightly) not building ESP32 binaries correctly due to uninitialized submodules** - Is it a bug fix, a new feature, a cleanup/refactor…? **CI Bug fix** ## Licensing confirmation (required) By checking the box below, you confirm ALL of the following: - You are the author of this contribution, or you have the right to contribute it. - You have read `CONTRIBUTING.md`. - You agree that this contribution may be merged, used, modified, and redistributed: - under the AGPLv3 Community Edition, **and** - under any proprietary / commercial / Enterprise editions of this project, now or in the future. - You understand that submitting this PR does not create any support obligation, SLA, or guarantee of merge. **I confirm the above licensing terms:** - [x] Yes, I agree ## Anything else? Optional: mention known limitations, follow-ups, or if this is related to an existing Issue.

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin gris-gris/fix-autobuild:gris-gris/fix-autobuild

git checkout gris-gris/fix-autobuild

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: dearsky/pico-hsm#124