Skip to content

Conversation

@sairon
Copy link
Member

@sairon sairon commented Sep 30, 2025

RaspberryMatic was renamed to OpenCCU in OpenCCU/OpenCCU#3162. This caused change of the name of the directory in the source tarball, causing build failure when the archive wasn't cached.

Summary by CodeRabbit

  • Chores
    • Switched the RPI-RF-MOD package to a new official upstream repository, aligning with current project ownership.
    • Updated the package integrity checksum to match the new tarball; license hash remains unchanged.
    • No version change or functional differences expected. Existing installations are unaffected; future builds use the new source.
    • Improves maintainability and ensures continued availability of package sources.

RaspberryMatic was renamed to OpenCCU in
OpenCCU/OpenCCU#3162. This caused change of the name of
the directory in the source tarball, causing build failure when the archive
wasn't cached.
@sairon sairon requested a review from agners September 30, 2025 10:30
@sairon sairon added the build Build and CI related issues label Sep 30, 2025
@coderabbitai
Copy link

coderabbitai bot commented Sep 30, 2025

📝 Walkthrough

Walkthrough

Updated the rpi-rf-mod package metadata: switched the source repository URL in the Buildroot makefile to OpenCCU/OpenCCU while keeping the version unchanged; updated the corresponding tarball sha256 in the hash file. No other entries or licenses modified.

Changes

Cohort / File(s) Summary of changes
Source repository update
buildroot-external/package/rpi-rf-mod/rpi-rf-mod.mk
Changed RPI_RF_MOD_SITE from GitHub repo jens-maus/RaspberryMatic to OpenCCU/OpenCCU; version and license fields unchanged.
Checksum update
buildroot-external/package/rpi-rf-mod/rpi-rf-mod.hash
Updated sha256 for rpi-rf-mod tarball to match new source; LICENSE hash unchanged; tarball filename unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title clearly and concisely describes the primary change—resolving a build failure due to the RaspberryMatic project being renamed—and directly reflects the pull request’s intent without extraneous details or ambiguity.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-rpi-rf-mod-rename

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
🧪 Early access (Sonnet 4.5): enabled

We are currently testing the Sonnet 4.5 model, which is expected to improve code review quality. However, this model may lead to increased noise levels in the review comments. Please disable the early access features if the noise level causes any inconvenience.

Note:

  • Public repositories are always opted into early access features.
  • You can enable or disable early access features from the CodeRabbit UI or by updating the CodeRabbit configuration file.

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

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
buildroot-external/package/rpi-rf-mod/rpi-rf-mod.mk (1)

10-10: Update comment URL to reflect new repository.

The comment still references the old RaspberryMatic URL pattern. Consider updating it to match the new OpenCCU repository structure.

Apply this diff to update the comment:

-# https://codeload.github.com/jens-maus/RaspberryMatic/tar.gz/COMMIT
+# https://codeload.github.com/OpenCCU/OpenCCU/tar.gz/COMMIT
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to data retention organization setting

📥 Commits

Reviewing files that changed from the base of the PR and between 435714c and 18e5607.

📒 Files selected for processing (2)
  • buildroot-external/package/rpi-rf-mod/rpi-rf-mod.hash (1 hunks)
  • buildroot-external/package/rpi-rf-mod/rpi-rf-mod.mk (1 hunks)
🔇 Additional comments (3)
buildroot-external/package/rpi-rf-mod/rpi-rf-mod.mk (2)

13-13: Repository URL updated correctly to OpenCCU.

The repository references have been properly updated from RaspberryMatic to OpenCCU, aligning with the project rename referenced in OpenCCU/OpenCCU#3162.

Also applies to: 18-18


44-51: Tarball structure matches expected directory and files. Confirmed that buildroot-external/package/rpi-rf-mod/dts/ and all referenced .dts files are present in the tarball.

buildroot-external/package/rpi-rf-mod/rpi-rf-mod.hash (1)

2-2: Manual SHA256 verification required

The sandbox environment lacks sha256sum, so the automated check failed. Please verify the SHA256 of the tarball manually in your local environment, for example:

curl -sL https://codeload.github.com/OpenCCU/OpenCCU/tar.gz/0ee62f7a87d26e990fb0e03b00befb505a7e44fd \
  | shasum -a 256

Confirm the output matches e1ea74165cd850518284dcc8ee394231aa7684169b752be67604838912fb9f23.

@sairon sairon merged commit cb66fa8 into dev Sep 30, 2025
3 checks passed
@sairon sairon deleted the fix-rpi-rf-mod-rename branch September 30, 2025 10:40
@github-actions github-actions bot locked and limited conversation to collaborators Oct 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

build Build and CI related issues cla-signed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants