[Bug 2335293] Review Request: rust-notify-types - Types used by the notify crate

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=2335293



--- Comment #5 from Ben Beasley <code@xxxxxxxxxxxxxxxxxx> ---
(In reply to Fabio Valentini from comment #4)
> It looks like one included file still has the old license header:
> src/event.rs
> 
> This references the old CC0-1.0 / Artistic-ish license of notify < 6, so
> it's probably a mistake, but it would be good to get confirmation.
> 
> Other than that, the package LGTM.

Hmm, according to
https://github.com/notify-rs/notify/commit/08e74dae8e96fbd25704cdaa530ffc02f6d33039#diff-fec21ee7a34c09969f535791a7963a9f59b57d12291e50864e0bb465aeb0e787
this file was just moved over from the notify crate, so the header is probably
correct. (The actual notify crate wasn’t relicensed for 7.0.0; it’s still
CC0-1.0.)

I guess I’ll open an upstream PR to adjust the crate license from MIT OR
Apache-2.0 to (MIT OR Apache-2.0) AND (CC0-1.0 OR Artistic-2.0).


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2335293

Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202335293%23c5

-- 
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux