Arch / AUR: README install docs; packaging in recordly-aur#33
Arch / AUR: README install docs; packaging in recordly-aur#33webadderall merged 1 commit intowebadderall:mainfrom
Conversation
|
Looks good overall and I’m fine with merging this direction, especially since you’re taking manual AUR maintainership. Before merge, can you please: (1) replace the placeholder maintainer line in PKGBUILD with your actual maintainer info, and (2) install the upstream MIT app license into /usr/share/licenses/$pkgname/ (the packaging LICENSE file in this PR is not the app’s MIT license). Small nitpick: remove empty MimeType= and consider Icon=dev.recordly.app plus pinned sha256sums per release. |
bb0f230 to
7939f15
Compare
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Comment Tip Migrating from UI to YAML configuration.Use the |
|
@webadderall new style: i'll have a sister repo https://github.com/firtoz/recordly-aur here for aur mgmt? updated this PR to reflect that, so this repo will be the base and the aur one can pull from this one and update aur with a CRON |
|
@webadderall also added a local installation script, no more changes pending from me so ready to review 🙏 |
|
Before merge, you need to (1) replace the placeholder maintainer line in PKGBUILD with your actual maintainer info, (2) install the upstream MIT app license into /usr/share/licenses/$pkgname/ (the packaging LICENSE file in this PR is not the app’s MIT license), (3) remove empty MimeType= and (4) consider Icon=dev.recordly.app plus pinned sha256sums per release. This is from my previous comment |
|
Also, the readme documents a local source-package flow that does not exist |
d41dc4f to
6142461
Compare
|
@webadderall updated PR, simplified greatly, now everything about the AUR stuff will live at https://github.com/firtoz/recordly-aur/ |
|
Thanks for contributing towards Recordly |
Description
Adds user-facing Arch / Manjaro documentation in this repo (AUR badge + install section) so people can install via
yay -S recordly-bin.Packaging is not maintained in this tree. The canonical PKGBUILD,
.desktopfile, checksum automation, and AUR git pushes live in firtoz/recordly-aur. That keeps upstream free of per-release Arch chores (nopkgver/sha256sumsbumps here).The AUR package applies maintainer review items there, for example:
LICENSE.mdat the release tag) installed under/usr/share/licenses/$pkgname/Icon=dev.recordly.app, no emptyMimeType=sha256sums, updated byupdate-aur.ts(Bun) + scheduled GitHub ActionsLocal install from a self-built AppImage is documented in recordly-aur (
PKGBUILD.from-source), not vianpmscripts in this repo.Motivation
Arch users get a clear path: AUR helper + link to how the package is built. Maintainers own release sync in recordly-aur without asking upstream to touch packaging on every release.
Type of change
Related issues
None.
For upstream maintainers
Nothing required on each release for Arch beyond normal tag + GitHub Release (AppImage +
LICENSE.mdat the tag). recordly-aur automation consumes that.Screenshots / video
How to verify
yay -S recordly-binand recordly-aur.yay -S recordly-bin(or build from templates in recordly-aur, not frompackaging/archin this repo).Checklist