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

@zackeeszackees 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

coderabbitaiBot 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 nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly matches the main change: updating ACLPC_CHECKSUM after the FastLED repository transfer.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck 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 mainJul 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>
Sign up for freeto 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

@zackees