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: ASSIGNED
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: itp-rust-nu-command
TreeView+ depends on / blocked
 
Reported: 2023-05-31 16:08 UTC by Michel Alexandre Salim
Modified: 2023-07-30 17:34 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Type: ---
Embargoed:
decathorpe: fedora-review?


Attachments (Terms of Use)

Description Michel Alexandre Salim 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.


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