Bug 2211438 (review-rust-notify4) - Review Request: rust-notify4 - Cross-platform filesystem notification library
Summary: Review Request: rust-notify4 - Cross-platform filesystem notification library
Keywords:
Status: CLOSED NOTABUG
Alias: review-rust-notify4
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Fabio Valentini
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FE-DEADREVIEW itp-rust-nu-command
TreeView+ depends on / blocked
 
Reported: 2023-05-31 16:08 UTC by Michel Lind
Modified: 2024-01-04 00:45 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2024-01-04 00:45:46 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Michel Lind 2023-05-31 16:08:29 UTC
Spec URL: https://salimma.fedorapeople.org/rust-notify4.spec
SRPM URL: https://salimma.fedorapeople.org/rust-notify4-4.0.17-1.fc39.src.rpm

Description:
Cross-platform filesystem notification library.

Fedora Account System Username: salimma

Comment 1 Fabio Valentini 2023-06-06 19:56:55 UTC
CC0-1.0 is no longer an acceptable license for Fedora software - there's a grace period for existing packages, but no new ones can be added. Not sure how to handle compat packages, but I'd rather avoid adding *more* CC0-1.0 packages ... I see that upstream isn't happy with the debouncing changes that happened in notify v5/6 (which is now creatively licensed "CC0-1.0 OR Artistic-2.0" - WTF, this isn't Perl), but is there something we can do to avoid notify v4?

Comment 2 Fabio Valentini 2023-07-30 16:55:35 UTC
Reading the guidelines around CC0-1.0 licensed projects again, I think the "grandfathering" rule applies here, and the package would be fine.
I'm also packaging notify-debouncer-mini for gitui, which might help you.

Comment 3 Fabio Valentini 2023-07-30 17:34:21 UTC
Note that newer versions of nu-command depend on notify-debouncer-full instead of notify v4:
https://crates.io/crates/nu-command/0.83.0/dependencies

I already opened a review request for notify-debouncer-mini (after some legal review), and another one for notify-debouncer-full wouldn't be much additional work. Let me know if I can help with that.

Comment 4 Fabio Valentini 2023-12-04 15:59:52 UTC
Is this compat package still required? We now have notify v5 and v6 in Fedora, and notify-debouncer-mini packaged (though not yet notify-debouncer-full).

Comment 5 Package Review 2024-01-04 00:45:46 UTC
This is an automatic action taken by review-stats script.

The ticket submitter failed to clear the NEEDINFO flag in a month.
As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
we consider this ticket as DEADREVIEW and proceed to close it.


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