Bug 2180614 - Review Request: ktextaddons - Various text handling addons
Summary: Review Request: ktextaddons - Various text handling addons
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Neal Gompa
QA Contact: Fedora Extras Quality Assurance
URL: https://invent.kde.org/libraries/ktex...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-03-21 21:49 UTC by marcdeop
Modified: 2023-05-08 12:09 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-03-28 18:31:36 UTC
Type: ---
Embargoed:
ngompa13: fedora-review+


Attachments (Terms of Use)

Description marcdeop 2023-03-21 21:49:17 UTC
Spec URL: https://marcdeop.fedorapeople.org/ktextaddons.spec
SRPM URL: https://marcdeop.fedorapeople.org/ktextaddons-1.1.1-1.fc39.src.rpm
Description: Various text handling addons
Fedora Account System Username: marcdeop

Comment 1 Neal Gompa 2023-03-21 21:53:44 UTC
Taking this review.

Comment 2 Neal Gompa 2023-03-21 21:57:34 UTC
> License:       CC0-1.0

This is not allowed for code in Fedora. Please talk to upstream to fix it.

Comment 3 Neal Gompa 2023-03-21 21:59:25 UTC
(Ignore the FE-Legal thing, that was an error)

Comment 4 Jakub Kadlčík 2023-03-21 22:03:06 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/5691692
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2180614-ktextaddons/fedora-rawhide-x86_64/05691692-ktextaddons/fedora-review/review.txt

Please take a look if any issues were found.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 5 Richard Fontana 2023-03-22 02:12:43 UTC
If the CC0 material is limited to what's stated in the .reuse/dep5 file:
Files: autocorrection/src/autotests/data/* textgrammarcheck/grammalecte/autotests/data/*.json textgrammarcheck/languagetool/autotests/data/*.json

I'm comfortable saying that should be allowed as being "content" rather than code.

Comment 6 marcdeop 2023-03-22 07:26:44 UTC
The full list of files which contain CC0 is:

README.md.license
sanitizers.supp
texttranslator/translator/plugins/google/CMakeLists.txt
texttranslator/translator/plugins/lingva/CMakeLists.txt
texttranslator/translator/plugins/bing/CMakeLists.txt
texttranslator/translator/plugins/yandex/CMakeLists.txt
texttranslator/translator/plugins/deepl/CMakeLists.txt
texttranslator/translator/plugins/libretranslate/CMakeLists.txt
texttranslator/translator/tests/CMakeLists.txt
texttranslator/designer/CMakeLists.txt
texttranslator/CMakeLists.txt
texttranslator/Messages.sh
metainfo.yaml.license
autocorrection/src/tests/CMakeLists.txt
autocorrection/src/autotests/CMakeLists.txt
autocorrection/src/CMakeLists.txt
autocorrection/src/Messages.sh
autocorrection/CMakeLists.txt
textgrammarcheck/CMakeLists.txt
textgrammarcheck/Messages.sh
textedittexttospeech/Messages.sh


I don't ship any of those files. I am a bit curious about the CMakeLists.txt files though. Are those ok?

Comment 7 marcdeop 2023-03-22 16:03:54 UTC
Upstream main developer assures me he will change the license for the CMakeLists and .sh files this week.

Comment 8 marcdeop 2023-03-22 19:23:51 UTC
Files have been relicensed upstream: https://invent.kde.org/libraries/ktextaddons/-/commit/8869e3393217d8be82a8d9c8cdfeb133374cd663

Comment 9 marcdeop 2023-03-23 12:56:27 UTC
@rfontana : would you mind checking if the changes are enough? the commit for the 1.1 branch: https://invent.kde.org/libraries/ktextaddons/-/commit/12969971afe1b41d6fca9099d60e384768be3fb9

Thanks!

Comment 10 Richard Fontana 2023-03-23 14:36:30 UTC
(In reply to marcdeop from comment #9)
> @rfontana : would you mind checking if the changes are enough?
> the commit for the 1.1 branch:
> https://invent.kde.org/libraries/ktextaddons/-/commit/
> 12969971afe1b41d6fca9099d60e384768be3fb9
> 
> Thanks!

I will just comment on the CC0 issue. I don't see any problems here for the Fedora CC0 policy, but one of the remaining CC0 files is the README.md which is currently basically empty. We have classified CC0 as allowed for "content" and disallowed generally (in particular for code) but haven't yet dealt with the issue of CC0 documentation. A non-empty README.md file should probably be classified as documentation. The current Fedora approach to documentation licensing is that "allowed-documentation" licenses have to meet all the requirements for normal "allowed" licenses, which CC0 doesn't. If the upstream developer were to actually populate the README.md file we'd have to face the question whether to make an exception for CC0 for documentation. We already have other CC licenses (which have the same problemlatic language about patents) grandfathered in to the allowed-documentation policy but we were disinclined to add CC0 because we didn't think it was commonly used for documentation in repositories otherwise not generally licensed under CC0. 

So there's nothing that has to be done here for this branch, but this could be an issue for the future, which the upstream developer could make disappear by picking a different license (e.g. BSD-3-Clause) for the possibly-someday-nonempty README.md :)

Comment 11 marcdeop 2023-03-25 08:46:01 UTC
Updated spec file and srpm with a new bugfix release from upstream

Comment 12 Neal Gompa 2023-03-25 12:29:35 UTC
This is good now.

PACKAGE APPROVED.

Comment 13 Fedora Admin user for bugzilla script actions 2023-03-25 21:40:19 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/ktextaddons

Comment 14 Kevin Kofler 2023-05-08 12:09:22 UTC
IMHO, this package should be renamed to kf5-ktextaddons or ktextaddons-kf5, because there will undoubtedly be a KF6 version soon.


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