https://bugzilla.redhat.com/show_bug.cgi?id=2020307
Bug ID: 2020307 Summary: Review Request: tokodon - Kirigami-based mastodon client Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Assignee: nobody@fedoraproject.org Reporter: thunderbirdtr@fedoraproject.org QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
https://pagure.io/Tokodon_rpm/blob/main/f/Tokodon.spec https://pagure.io/Tokodon_rpm/blob/main/f/tokodon-21.08-1.fc35.src.rpm Description: Tokodon is a Mastodon client for Plasma and Plasma Mobile.
Pagure URL : https://pagure.io/Tokodon_rpm/tree/main
Fedora Account System Username: thunderbirdtr
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
Ankur Sinha (FranciscoD) sanjay.ankur@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Doc Type|--- |If docs needed, set a value CC| |sanjay.ankur@gmail.com
--- Comment #1 from Ankur Sinha (FranciscoD) sanjay.ankur@gmail.com --- Updating URLs to use the raw URL from pagure for fedora-review:
Spec: https://pagure.io/Tokodon_rpm/raw/main/f/Tokodon.spec SRPM: https://pagure.io/Tokodon_rpm/raw/main/f/tokodon-21.08-1.fc35.src.rpm
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
Onuralp SEZER thunderbirdtr@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Comment|0 |updated
--- Comment #0 has been edited ---
https://pagure.io/Tokodon_rpm/blob/main/f/tokodon.spec https://pagure.io/Tokodon_rpm/blob/main/f/tokodon-21.08-1.fc35.src.rpm Description: Tokodon is a Mastodon client for Plasma and Plasma Mobile.
Pagure URL : https://pagure.io/Tokodon_rpm/tree/main
Fedora Account System Username: thunderbirdtr
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
--- Comment #2 from Onuralp SEZER thunderbirdtr@fedoraproject.org --- Updated URL
Spec: https://pagure.io/Tokodon_rpm/raw/main/f/tokodon.spec
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
Arthur Bols arthur@bols.dev changed:
What |Removed |Added ---------------------------------------------------------------------------- Assignee|nobody@fedoraproject.org |arthur@bols.dev Status|NEW |ASSIGNED CC| |arthur@bols.dev
--- Comment #3 from Arthur Bols arthur@bols.dev --- - Please update to 21.12 Don't forget to also update the srpm.
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
Justin Zobel justin.zobel@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |justin.zobel@gmail.com
--- Comment #4 from Justin Zobel justin.zobel@gmail.com --- Have created a PR on Onuralp's repo for 21.12, rebuild srpm and add app icon that is installed.
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
Onuralp Sezer thunderbirdtr@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |thunderbirdtr@gmail.com
--- Comment #5 from Onuralp Sezer thunderbirdtr@gmail.com --- (In reply to Arthur Bols from comment #3)
- Please update to 21.12
Don't forget to also update the srpm.
New Spec : https://pagure.io/Tokodon_rpm/blob/main/f/tokodon.spec New SRPM : https://pagure.io/Tokodon_rpm/blob/main/f/tokodon-21.12-1.fc35.src.rpm
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
--- Comment #6 from Arthur Bols arthur@bols.dev --- - The spec file url should point to the raw file, can you fix that please? - Mock build failed:
CMake Error at CMakeLists.txt:52 (find_package): By not providing "FindKF5QQC2DesktopStyle.cmake" in CMAKE_MODULE_PATH this project has asked CMake to find a package configuration file provided by "KF5QQC2DesktopStyle", but CMake did not find one. Could not find a package configuration file provided by "KF5QQC2DesktopStyle" with any of the following names: KF5QQC2DesktopStyleConfig.cmake kf5qqc2desktopstyle-config.cmake Add the installation prefix of "KF5QQC2DesktopStyle" to CMAKE_PREFIX_PATH or set "KF5QQC2DesktopStyle_DIR" to a directory containing one of the above files. If "KF5QQC2DesktopStyle" provides a separate development package or SDK, be sure it has been installed.
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
--- Comment #7 from Onuralp SEZER thunderbirdtr@fedoraproject.org --- BR : KF5QQC2DesktopStyle added.
SPEC File : https://pagure.io/Tokodon_rpm/raw/main/f/tokodon.spec SRPM File : https://pagure.io/Tokodon_rpm/raw/main/f/tokodon-21.12-1.fc35.src.rpm
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
Arthur Bols arthur@bols.dev changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review?
--- Comment #8 from Arthur Bols arthur@bols.dev --- Again a licensing issue. :/
I'm in a hurry so here's a quick rundown of issues, but overall it looks quite good. - Add a license breakdown - Report the missing licenses upstream - Add the README.md with %doc
sed -i 's/SocialMedia/X-SocialMedia/g' %{buildroot}%{_kf5_datadir}/applications/org.kde.%{name}.desktop
You have a commit upstream to fix it. Please add it as a comment so you won't forget to remove it in the future. A patch would be better. I'm fine with it in this case, since it should be gone in the next release.
%license LICENSES/*.txt
You can remove the .txt to match all files. This would be useful if they add a license file without extension.
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
--- Comment #9 from Onuralp SEZER thunderbirdtr@fedoraproject.org --- (In reply to Arthur Bols from comment #8)
Again a licensing issue. :/
I'm in a hurry so here's a quick rundown of issues, but overall it looks quite good.
- Add a license breakdown
- Report the missing licenses upstream
- Add the README.md with %doc
sed -i 's/SocialMedia/X-SocialMedia/g' %{buildroot}%{_kf5_datadir}/applications/org.kde.%{name}.desktop
You have a commit upstream to fix it. Please add it as a comment so you won't forget to remove it in the future. A patch would be better. I'm fine with it in this case, since it should be gone in the next release.
%license LICENSES/*.txt
You can remove the .txt to match all files. This would be useful if they add a license file without extension.
All done and that "sed" fix already merged as well (https://invent.kde.org/network/tokodon/-/merge_requests/21) we just gonna wait next version and gonna remove that as well.
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
--- Comment #10 from Onuralp SEZER thunderbirdtr@fedoraproject.org --- I should check licenses more carefully but we would like to push all the plasma-mobile related packages a bit quickly and then "polish" it afterwards (License sections and be fix easily as well yet again it I should be more careful and check a bit more details about it)
Also thank you for reviews
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
Justin Zobel justin.zobel@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Assignee|arthur@bols.dev |justin.zobel@gmail.com
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
--- Comment #11 from Justin Zobel justin.zobel@gmail.com --- - Update to the latest version (fixes the desktop file issue so that sed can be removed)
- Add licenses: https://invent.kde.org/network/tokodon/-/blob/master/src/content/ui/Settings... is licensed under GPLv2+ https://invent.kde.org/network/tokodon/-/blob/master/src/content/ui/Settings... is licensed under LGPL-2.1-or-later https://invent.kde.org/network/tokodon/-/blob/master/src/content/ui/Settings... is licensed under LGPL-2.0-or-later
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
--- Comment #12 from Onuralp Sezer thunderbirdtr@gmail.com --- All done
SPEC : https://pagure.io/Tokodon_rpm/raw/main/f/tokodon.spec SRPM : https://pagure.io/Tokodon_rpm/raw/main/f/tokodon-22.02-1.fc36.src.rpm
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
--- Comment #13 from Justin Zobel justin.zobel@gmail.com --- - Licenses have been included in the package but not mentioned in the spec License field.
- See https://invent.kde.org/network/tokodon/-/tree/master/LICENSES for the ones you might be missing or below for full licensecheck.txt
- Fix up that and we should be good to go!
*No copyright* Creative Commons Attribution 4.0 ----------------------------------------------- tokodon-22.02/README.md
*No copyright* Creative Commons CC0 1.0 --------------------------------------- tokodon-22.02/.gitlab-ci.yml tokodon-22.02/.kde-ci.yml tokodon-22.02/LICENSES/CC0-1.0.txt tokodon-22.02/android/AndroidManifest.xml tokodon-22.02/org.kde.tokodon.appdata.xml tokodon-22.02/org.kde.tokodon.desktop tokodon-22.02/src/Messages.sh
*No copyright* GNU General Public License, Version 2 ---------------------------------------------------- tokodon-22.02/src/content/ui/Settings/AccountsPage.qml
*No copyright* GNU General Public License, Version 3 ---------------------------------------------------- tokodon-22.02/src/account.cpp tokodon-22.02/src/account.h tokodon-22.02/src/accountmanager.cpp tokodon-22.02/src/accountmanager.h tokodon-22.02/src/accountmodel.cpp tokodon-22.02/src/accountmodel.h tokodon-22.02/src/attachmenteditormodel.cpp tokodon-22.02/src/attachmenteditormodel.h tokodon-22.02/src/clipboard.cpp tokodon-22.02/src/clipboard.h tokodon-22.02/src/content/ui/AccountInfo.qml tokodon-22.02/src/content/ui/AuthorizationPage.qml tokodon-22.02/src/content/ui/FullScreenImage.qml tokodon-22.02/src/content/ui/LoginPage.qml tokodon-22.02/src/content/ui/MastoPage.qml tokodon-22.02/src/content/ui/TimelinePage.qml tokodon-22.02/src/content/ui/TootComposer.qml tokodon-22.02/src/content/ui/main.qml tokodon-22.02/src/main.cpp tokodon-22.02/src/notificationmodel.cpp tokodon-22.02/src/notificationmodel.h tokodon-22.02/src/post.cpp tokodon-22.02/src/post.h tokodon-22.02/src/threadmodel.cpp tokodon-22.02/src/threadmodel.h tokodon-22.02/src/timelinemodel.cpp tokodon-22.02/src/timelinemodel.h
*No copyright* GNU Lesser General Public License, Version 2.1 ------------------------------------------------------------- tokodon-22.02/src/config.kcfgc tokodon-22.02/src/content/ui/Settings/GeneralPage.qml tokodon-22.02/src/content/ui/Settings/SonnetConfigPage.qml
*No copyright* GNU Library General Public License, Version 2.0 -------------------------------------------------------------- tokodon-22.02/src/content/ui/Settings/SettingsPage.qml
Unknown or generated -------------------- tokodon-22.02/CMakeLists.txt tokodon-22.02/LICENSES/GPL-3.0-only.txt tokodon-22.02/org.kde.tokodon.svg tokodon-22.02/po/ca/tokodon.po tokodon-22.02/po/ca@valencia/tokodon.po tokodon-22.02/po/cs/tokodon.po tokodon-22.02/po/de/tokodon.po tokodon-22.02/po/en_GB/tokodon.po tokodon-22.02/po/es/tokodon.po tokodon-22.02/po/fi/tokodon.po tokodon-22.02/po/fr/tokodon.po tokodon-22.02/po/hu/tokodon.po tokodon-22.02/po/it/tokodon.po tokodon-22.02/po/ko/tokodon.po tokodon-22.02/po/nl/tokodon.po tokodon-22.02/po/pl/tokodon.po tokodon-22.02/po/pt/tokodon.po tokodon-22.02/po/pt_BR/tokodon.po tokodon-22.02/po/sk/tokodon.po tokodon-22.02/po/sl/tokodon.po tokodon-22.02/po/sv/tokodon.po tokodon-22.02/po/uk/tokodon.po tokodon-22.02/po/zh_CN/tokodon.po tokodon-22.02/src/CMakeLists.txt tokodon-22.02/src/config.kcfg tokodon-22.02/src/content/elephant.svg tokodon-22.02/src/content/icon/boost-post-done.svg tokodon-22.02/src/content/icon/boost-post.svg tokodon-22.02/src/content/icon/like-post-done.svg tokodon-22.02/src/content/icon/like-post.svg tokodon-22.02/src/resources.qrc
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
Justin Zobel justin.zobel@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |POST Flags|fedora-review? |fedora-review+
--- Comment #14 from Justin Zobel justin.zobel@gmail.com --- Approved.
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
--- Comment #15 from Onuralp Sezer thunderbirdtr@gmail.com --- SCMs links are created
https://pagure.io/releng/fedora-scm-requests/issue/43741 https://pagure.io/releng/fedora-scm-requests/issue/43742 https://pagure.io/releng/fedora-scm-requests/issue/43743
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
--- Comment #16 from Gwyn Ciesla gwync@protonmail.com --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/tokodon
https://bugzilla.redhat.com/show_bug.cgi?id=2020307
Justin Zobel justin.zobel@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|POST |CLOSED Resolution|--- |CURRENTRELEASE Last Closed| |2023-02-06 07:40:32
package-review@lists.fedoraproject.org