[Bug 1645765] Review Request: slurp - Select a region in a Wayland compositor

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

 



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



--- Comment #3 from Jan Pokorný <jpokorny@xxxxxxxxxx> ---
Sorry for my blindness to "issues" section, was concentrating merely
on the checkboxes since my brain was learnt that there's usually a lot
of ballast coming from fedora-review (meaning it likely needs some
love to catch up the evolution):

> - The file protocol/wlr-layer-shell-unstable-v1.xml is licensed with
>   "NTP (legal disclaimer)". It's not on the list of good licenses. You
>   should check with Fedora Legal whether this is a good license.

Indeed it is a good license, and that's just unfortunate than
licensecheck provides a misleading verdict vs. how Fedora classifies
the license -- this is the matching text:

https://fedoraproject.org/wiki/Licensing:MIT#Old_Style_with_legal_disclaimer

but it contains "and sell" addition to the very first sentences, but
this matches with, e.g.,

https://fedoraproject.org/wiki/Licensing:MIT#Old_Style

(see also [bug 1475962 comment 12]).

Do you see any conflict here?  I shall fix the wording in the spec to
this very interpretation since otherwise I am carrying this cargo cult
confusion (originating in wlroots since [bug 1529352 comment 5]),
though.

And you should likely do the same with sway, since the protocol file
in question was borrowed there :-)

> - Upstream files are not properly licensed, most files are missing
>   license headers. This should at least be reported to upstream.

Cannot find anyting to that effect in
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/LicensingGuidelines
Can you point out the particular part to me that asks for this, please?

(Again, sway is in the same boat here.)

My TODO is to to get the slurp-grim.desktop more into shape
(especially Categories).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




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

  Powered by Linux