nextcloud / nextcloud/notify_push
Build and include notify_push for FreeBSD in every release too
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 344
- Forks
- 64
- Avg merge
- 2d 10h
- Merged PRs (30d)
- 5
Description
Is your feature request related to a problem? Please describe.
Nextcloud notify_push is great, but it is included for Linux aarch64, armv7 and x86_64 only. FreeBSD is missing, while it can easily be included in the Nextcloud release process. I feel a bit frustrated to build notify_push and testing with occ with every Netxcloud update myself; mundane work that can be automated to benefit all FreeBSD admins.
Describe the solution you'd like
Include fbsd_amd64 or fbsd_x86_64 in directory:
/NEXTCLOUD_ROOT/apps/notify_push/bin/
Execute in the release build process (generic FreeBSD fits-all):
cargo build --release --target=x86_64-unknown-freebsd
Describe alternatives you've considered
Manually running cargo build --release --target=x86_64-unknown-freebsd, moving compiled push_notify to correct location and re-running occ to check whether all is ok over-and-over again every update.
Additional context
No other context. Except a "thank you" for considering!
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the release build process and the /NEXTCLOUD_ROOT/apps/notify_push/bin/ directory. Run cargo build --release --target=x86_64-unknown-freebsd and verify that the resulting FreeBSD binary is included in every release at the expected path and works with occ.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- build-system, release
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 62/100