Bug 2211438 (review-rust-notify4)

Summary:	Review Request: rust-notify4 - Cross-platform filesystem notification library
Product:	[Fedora] Fedora	Reporter:	Michel Alexandre Salim <michel>
Component:	Package Review	Assignee:	Fabio Valentini <decathorpe>
Status:	ASSIGNED ---	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	decathorpe, package-review
Target Milestone:	---	Flags:	decathorpe: fedora-review?
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	2211172

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.