Skip to content

fix(packages): update ACLPC_CHECKSUM after FastLED org transfer - #915

Merged
zackees merged 1 commit into
mainfrom
fix/lpc8xx-checksum-and-port-api
Jul 1, 2026
Merged

fix(packages): update ACLPC_CHECKSUM after FastLED org transfer#915
zackees merged 1 commit into
mainfrom
fix/lpc8xx-checksum-and-port-api

Conversation

@zackees

@zackees zackees commented Jul 1, 2026

Copy link
Copy Markdown
Member

Problem

The `FastLED/framework-arduino-lpc8xx` repository was org-transferred from `zackees/` to `FastLED/` on 2026-06-28. GitHub regenerates archive metadata after ownership changes (the archive's top-level directory name flips owner in the tarball prefix), so the SHA256 of the archive at

https://github.com/FastLED/framework-arduino-lpc8xx/archive/50d76e0d63c2d2a2b365b29de47100d93c530c83.tar.gz

changed even though the git tree is byte-identical. Every `bash compile lpc845brk` on the FastLED side now aborts with:

```
build error: package error: checksum mismatch for
...framework-arduino-lpc8xx...
50d76e0d63c2d2a2b365b29de47100d93c530c83.tar.gz:
expected e6bbcc3392ae9da44c5c92268d7cd953770e0011fcf285561104633e36087428,
got e64f0cb81ab1673e05c2a63b7b4548c1373c9ba885413d2a2351bbfae160fbec
```

Fix

Update `ACLPC_CHECKSUM` to the current archive SHA256:

e64f0cb81ab1673e05c2a63b7b4548c1373c9ba885413d2a2351bbfae160fbec

Verification

Manually confirmed on 2026-07-01 against a fresh fbuild-cache download. With this fix (applied via binary-patch of the installed fbuild.exe to prove the checksum-only change works), `bash compile lpc845brk --examples AutoResearch --build-flag "-DFASTLED_LPC_PWM_DMA=1"` on FastLED master completes end-to-end:

```
Board: NXP LPC845-BRK / LPC845 @ 24MHz
Toolchain: arm-none-eabi-gcc 15.2.1
Flash: 53.52KB / 64.00KB (83.6%)
RAM: 9.07KB / 16.00KB (56.7%)
build succeeded in 212.2s
```

Refs

  • One of the blockers to on-silicon SCT+DMA channels-API validation (FastLED#3468).
  • FastLED#3450 (LPC restoration meta) B2.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated the verified download checksum for the Arduino LPC8xx framework to match the latest upstream archive.
    • Refreshed the accompanying notes to reflect the upstream repository transfer and verification details.

The `FastLED/framework-arduino-lpc8xx` repository was org-transferred
from `zackees/` to `FastLED/` on 2026-06-28. GitHub regenerates archive
metadata after ownership changes (the archive's top-level directory
name flips owner in the tarball prefix), so the SHA256 of the archive
at

    https://github.com/FastLED/framework-arduino-lpc8xx/archive/
    50d76e0d63c2d2a2b365b29de47100d93c530c83.tar.gz

changed even though the git tree is byte-identical. Every `bash
compile lpc845brk` on the FastLED side now aborts with

    build error: package error: checksum mismatch for
      ...framework-arduino-lpc8xx...
      50d76e0d63c2d2a2b365b29de47100d93c530c83.tar.gz:
      expected e6bbcc3392ae9da44c5c92268d7cd953770e0011fcf285561104633e36087428,
      got      e64f0cb81ab1673e05c2a63b7b4548c1373c9ba885413d2a2351bbfae160fbec

Update `ACLPC_CHECKSUM` to `e64f0cb81ab1673e05c2a63b7b4548c1373c9ba885413d2a2351bbfae160fbec`
(verified against the archive GitHub currently serves).

Manually confirmed on 2026-07-01:

    $ sha256sum <fresh fbuild cache download of the tarball>
    e64f0cb81ab1673e05c2a63b7b4548c1373c9ba885413d2a2351bbfae160fbec

With this checksum, `bash compile lpc845brk --examples AutoResearch
--build-flag "-DFASTLED_LPC_PWM_DMA=1"` completes end-to-end on
FastLED master (53.52 KB / 64 KB flash, 9.07 KB / 16 KB RAM) — one of
the blockers to on-silicon SCT+DMA channels-API validation
(FastLED #3468).

Refs FastLED #3450 (LPC restoration meta) B2.
@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 508d9749-99cc-437a-a8b6-5026a5abca82

📥 Commits

Reviewing files that changed from the base of the PR and between f5219dc and 26f71e4.

📒 Files selected for processing (1)
  • crates/fbuild-packages/src/library/arduino_core_lpc8xx.rs

📝 Walkthrough

Walkthrough

The pinned SHA256 checksum constant ACLPC_CHECKSUM for the framework-arduino-lpc8xx archive is updated in arduino_core_lpc8xx.rs, along with an accompanying comment documenting the upstream repository ownership transfer and verification details.

Changes

Checksum Update

Layer / File(s) Summary
Update pinned archive checksum
crates/fbuild-packages/src/library/arduino_core_lpc8xx.rs
ACLPC_CHECKSUM constant value updated to a new SHA256, with comment documenting the GitHub archive ownership transfer and verification date/SHA.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Related PRs: None found

Suggested labels: dependencies, low-risk

Suggested reviewers: None found

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: updating ACLPC_CHECKSUM after the FastLED repository transfer.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/lpc8xx-checksum-and-port-api

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@zackees
zackees merged commit d08622b into main Jul 1, 2026
86 of 93 checks passed
@zackees
zackees deleted the fix/lpc8xx-checksum-and-port-api branch July 1, 2026 06:47
zackees added a commit that referenced this pull request Jul 1, 2026
…#924)

Cargo.toml was bumped to 2.3.16 in c99579e but pyproject.toml stayed at
2.3.15, causing every release-auto run since (ecd80c7, dbc2245) to
fail in the 'Prepare release' step's Cargo/pyproject lockstep check.

Bump pyproject.toml so the release fires. Downstream: FastLED's
'fbuild==2.3.15' pin will pick up 2.3.16 on the next 'uv sync',
which includes the FastLED-org-transfer ACLPC_CHECKSUM fix from #915
(d08622b). That unblocks the LPC804/LPC845/lpcxpresso804/lpcxpresso845max
build lanes on FastLED, which have been red with 'sha256 mismatch
for FastLED-framework-arduino-lpc8xx/.../50d76e0d63c...tar.gz'
since GitHub's post-transfer tarball rebuild changed the archive
bytes for the same source commit.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@fastled-project-sync fastled-project-sync Bot moved this to Triage in FastLED Tracker Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

1 participant