Skip to content

Conversation

brentru
Copy link
Member

@brentru brentru commented Sep 23, 2024

Motivation
This pull request simplifies our process for building WipperSnapper Arduino and allows us to:

  1. Not build WipperSnapper from arduino-esp32 release, by default
  2. Apply patches to a fork/branch of the arduino-esp32 bsp. This will allow for us to stop waiting for a new release and patch/rebuild WS directly.
  3. Build specific platforms (i.e: FunHouse ESP32-S2) on specific BSP versions (or patched BSP versions)

Technical Details
Instead of using adafruit/ci-arduino, which pulls from the latest arduino-esp32 release, this PR points build-clang-doxy.yml to the ci-wippersnapper branch of adafruit/ci-arduino.

Within this branch is a modified all_platforms.py file which:

  1. Points to a branch of adafruit/arduino-esp32 corresponding to a release of the espressif/arduino-esp32.

For example, the all_platforms.py targets currently point to the repo adafruit/arduino-esp32 and branch wipper-bsp-3.0.5-idf-5.1.4. This branch is wippersnapper-specific (denoted by the wipper- prefix), builds on espressif/arduino-esp32's release 3.0.5 (bsp-3.0.5), and uses ESP-IDF 5.1.4 (idf-5.1.4).

  1. Prefixes the platform's FQBN with espressif:, allowing it to build from source rather than a pre-packaged release.

@ladyada - As a result of this PR and tooling changes, you may see some new branches across adafruit/arduino-esp32 (i.e: https://github.com/adafruit/arduino-esp32/tree/wipper-bsp-3.0.5-idf-5.1.4) to track espressif's releases. We are also using the ci-wippersnapper branch of adafruit/ci-arduino as a build script for WipperSnapper.

@brentru brentru requested a review from tyeth September 23, 2024 18:57
Copy link
Member

@tyeth tyeth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@brentru brentru merged commit 47da594 into main Sep 23, 2024
33 checks passed
@brentru brentru deleted the update-bcd-to-use-adafruit-ci branch September 23, 2024 19:44
@ladyada
Copy link
Member

ladyada commented Sep 26, 2024

thanks good idea!

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.

3 participants