Skip to content

Conversation

@wentywenty
Copy link

@wentywenty wentywenty commented Aug 21, 2025

  • [√ ] Use conventional PR title: <manifest-name[@version]|chore>: <general summary of the pull request>
  • [ √] I have read the Contributing Guide

Summary by CodeRabbit

  • New Features
    • Added a Device Tree Compiler (dtc) v1.7.2-1 package manifest, providing a complete toolchain for working with device tree source and binary files. Includes 64‑bit and 32‑bit builds and command‑line utilities for compiling and inspecting device tree binaries, plus automated version checks and update metadata.

@github-actions
Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

dtc

  • Description
  • License
  • Hashes

@wentywenty
Copy link
Author

/verify

@github-actions
Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

dtc

  • Description
  • License
  • Hashes

@wentywenty wentywenty changed the title Add [email protected] dtc: Add version 1.7.2-1 Aug 21, 2025
@aliesbelik aliesbelik added the package-request-needed Create a package request issue before raising PR. Check the criteria for a package to be accepted. label Aug 22, 2025
@z-Fng z-Fng marked this pull request as draft October 19, 2025 12:52
@wentywenty
Copy link
Author

#16421

@wentywenty wentywenty marked this pull request as ready for review October 21, 2025 02:48
@coderabbitai
Copy link

coderabbitai bot commented Oct 21, 2025

Warning

Rate limit exceeded

@wentywenty has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 4 minutes and 45 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 03fcb39 and b72cf9c.

📒 Files selected for processing (1)
  • bucket/dtc.json (1 hunks)

Walkthrough

Adds a new package manifest bucket/dtc.json for Device Tree Compiler (dtc) v1.7.2-1 with metadata, license, architecture-specific downloads and checks, and four executable bins. (44 words)

Changes

Cohort / File(s) Summary
Package Manifest
bucket/dtc.json
New manifest for Device Tree Compiler (dtc) v1.7.2-1: metadata (description, homepage), license GPL-2.0-only, architecture-specific download URLs and SHA256 hashes for 64bit/32bit, bin entries (dtc.exe, fdtdump.exe, fdtget.exe, fdtput.exe), checkver and autoupdate rules.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A manifest arrived, neat and bright,
Binaries tucked in, hashes tight,
Version penned with a careful hop,
Links all set — no need to stop,
I nibble joy and dance tonight.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The PR title "dtc: Add version 1.7.2-1" directly and accurately describes the primary change in the changeset, which is adding a new manifest file for the Device Tree Compiler package at version 1.7.2-1. The title is concise, clear, and follows the conventional format specified in the repository's contributing guidelines. It would be immediately understood by a teammate scanning the project history.
Description Check ✅ Passed The PR description includes the template structure with the template comments preserved and contains the required checklist items, both of which are marked as completed by the author. The author has indicated they used the conventional PR title format and read the Contributing Guide as required. While the issue reference section (Closes/Relates to) is not filled in, the template structure shows this as optional with an "or" clause, and the mandatory checklist items are properly addressed.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

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 and usage tips.

@wentywenty
Copy link
Author

/verify

@github-actions
Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

dtc

  • Lint
  • Description
  • License
  • Hashes

Check the full log for details.

@aliesbelik aliesbelik linked an issue Oct 21, 2025 that may be closed by this pull request
6 tasks
@aliesbelik aliesbelik removed the package-request-needed Create a package request issue before raising PR. Check the criteria for a package to be accepted. label Oct 21, 2025
@aliesbelik aliesbelik marked this pull request as draft October 21, 2025 03:15
@wentywenty wentywenty marked this pull request as ready for review October 21, 2025 03:18
@wentywenty
Copy link
Author

May I ask what's wrong?

Copy link
Collaborator

@aliesbelik aliesbelik left a comment

Choose a reason for hiding this comment

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

  1. Add checkver/autoupdate sections.
  2. Add architecture.64bit wrapper.

@wentywenty
Copy link
Author

/verify

@github-actions
Copy link
Contributor

Your changes did not pass all checks.

Please address the issues in the manifest and comment starting with /verify to rerun the checks.

dtc

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Check the full log for details.

@wentywenty
Copy link
Author

/verify

@github-actions
Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

dtc

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Check the full log for details.

@wentywenty
Copy link
Author

  1. Add checkver/autoupdate sections.
  2. Add architecture.64bit wrapper.

i fix it

@wentywenty
Copy link
Author

/verify

@github-actions
Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

dtc

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Check the full log for details.

@wentywenty
Copy link
Author

i`m ready

@z-Fng
Copy link
Member

z-Fng commented Jan 6, 2026

Closing this as per #16421 (comment).

@z-Fng z-Fng closed this Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Request]: dtc

3 participants