Skip to content

update the pkgbuild des and version#32

Merged
moheladwy merged 2 commits intomainfrom
update-the-pkgdec
Mar 21, 2026
Merged

update the pkgbuild des and version#32
moheladwy merged 2 commits intomainfrom
update-the-pkgdec

Conversation

@moheladwy
Copy link
Copy Markdown
Owner

No description provided.

@moheladwy moheladwy self-assigned this Mar 21, 2026
@moheladwy moheladwy added documentation Improvements or additions to documentation enhancement New feature or request labels Mar 21, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 21, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 70e49d92-97fb-4451-be04-935398c196a1

📥 Commits

Reviewing files that changed from the base of the PR and between 74cd888 and 576ea03.

📒 Files selected for processing (1)
  • PKGBUILD

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Package versioning now derived from repository commit metadata (dynamic build-aware versioning).
    • Package description explicitly mentions support for both X11 and Wayland display servers.
    • Added a package identity alias ("provides") for compatibility.
    • Deterministic packaging layout and explicit license placement under standard license path.

Walkthrough

PKGBUILD changed to use Git-derived dynamic pkgver, pkgdesc updated to mention x11 and wayland, provides=('OCR4Linux') added, package() now cds to ${srcdir}/${pkgname}-${pkgver}, and license install path updated to include LICENSE filename.

Changes

Cohort / File(s) Summary
PKGBUILD Configuration
PKGBUILD
Replaced static pkgver with a pkgver() that derives 1.4.2.r<count>.<short_sha> from Git; updated pkgdesc to mention "x11 and wayland"; added provides=('OCR4Linux'); simplified package() to cd "${srcdir}/${pkgname}-${pkgver}" (removed dynamic subdir discovery); install path for license now targets ${pkgdir}/usr/share/licenses/${pkgname}/LICENSE.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 I hop through commits, counting each head,
Git-ticked version jingles where static once tread,
X11 and Wayland both in my tune,
LICENSE tucked neat beneath the moon,
A package reborn — a cheerful little thrum.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update-the-pkgdec

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.

@moheladwy moheladwy merged commit 2b3edc3 into main Mar 21, 2026
5 checks passed
@moheladwy moheladwy deleted the update-the-pkgdec branch March 21, 2026 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant