Bug 2362060 - Review Request: niri - Scrollable-tiling Wayland compositor
Summary: Review Request: niri - Scrollable-tiling Wayland compositor
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Neal Gompa
QA Contact: Fedora Extras Quality Assurance
URL: https://github.com/YaLTeR/niri
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2025-04-24 14:13 UTC by Fabio Valentini
Modified: 2025-05-20 13:17 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2025-05-20 13:17:47 UTC
Type: ---
Embargoed:
ngompa13: fedora-review+


Attachments (Terms of Use)

Description Fabio Valentini 2025-04-24 14:13:22 UTC
Spec URL: https://decathorpe.fedorapeople.org/niri.spec
SRPM URL: https://decathorpe.fedorapeople.org/niri-25.02-1.fc42.src.rpm

Description:
A scrollable-tiling Wayland compositor.

Fedora Account System Username: decathorpe

Comment 1 Fabio Valentini 2025-04-24 14:13:25 UTC
This package built on koji:  https://koji.fedoraproject.org/koji/taskinfo?taskID=131937050

Comment 2 Fabio Valentini 2025-04-24 15:40:29 UTC
Note: This package builds with vendored dependencies, in part based on how the packages for COSMIC work since they share some similarities. Notably, the dependency on smithay (the Wayland compositor library) is a git snapshot, which would need to be vendored anyway.

Comment 3 Fedora Review Service 2025-04-24 15:40:33 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/8963813
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2362060-niri/fedora-rawhide-x86_64/08963813-niri/fedora-review/review.txt

Found issues:

- Systemd user unit service file(s) in niri
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/Scriptlets/#_user_units

Please know that there can be false-positives.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 4 Neal Gompa 2025-04-25 04:38:05 UTC
Taking this review.

Comment 5 Neal Gompa 2025-05-16 02:51:43 UTC
Package was generated with rust2rpm, simplifying the review.

✅ package contains only permissible content
✅ package builds and installs without errors on rawhide
✅ latest version packaged
✅ license matches upstream specification and is acceptable for Fedora
✅ license file is included with %license in %files
✅ package complies with Rust Packaging Guidelines (using vendored crates for now)

Note: consider seeing if using wlheadless-run lets you run the test suite. Not blocking this on that though.

Package APPROVED.

===

Recommended post-import tasks:

- set up package on release-monitoring.org:

- add @rust-sig with "commit" access as package co-maintainer
  (should happen automatically)

- track package in koschei for all built branches
  (should happen automatically once rust-sig is co-maintainer)

Comment 6 Fabio Valentini 2025-05-19 22:11:36 UTC
Thank you for the review!

I forgot about xwayland-run et al, I'll take a look if it can be used to run the tests.

I'll look into updating to 25.05 too, it was released a few days ago and looks like a great release.

Comment 7 Fedora Admin user for bugzilla script actions 2025-05-19 22:12:41 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/niri

Comment 8 Fabio Valentini 2025-05-20 13:17:47 UTC
Imported, updated to 25.05, and built:
https://bodhi.fedoraproject.org/updates/FEDORA-2025-4ec8af6ae3


Note You need to log in before you can comment on or make changes to this bug.