Spec URL: http://acksyn.org/files/rpms/polly/polly.spec SRPM URL: http://acksyn.org/files/rpms/polly/polly-0.93.11-1.fc20.src.rpm Description: Polly is a Linux Twitter client designed for managing multiple columns of multiple accounts, purposefully keeping them separated instead of merged. Fedora Account System Username: mbaldessari Note: There was a previous review request where the original reporter lost interest - https://bugzilla.redhat.com/show_bug.cgi?id=824218 As suggested there in c#17
As a quickstart, run "fedora-review -b 1026042" and examine its various results. > Summary:Linux Twitter client designed for multiple columns > of multiple accounts Linux, eh? ;-) > desktop-file-install \ > … > desktop-file-validate Note that desktop-file-install also validates the file, so you only need to run either one, not both. > %{_datadir}/icons/*/*/*/* If the package doesn't "Requires: hicolor-icon-theme", there are multiple unowned directories in the /usr/share/icons/ tree. https://fedoraproject.org/wiki/Packaging:Guidelines#File_and_Directory_Ownership https://fedoraproject.org/wiki/Packaging:UnownedDirectories > %{_datadir}/%{name}/iconsets Here the directory %{_datadir}/%{name} is not included.
Holy Batman! That was fast ;) (In reply to Michael Schwendt from comment #1) > As a quickstart, run "fedora-review -b 1026042" and examine its various > results. Ha! Good to know, I was a bit concerned that it did other admin stuff as well. > > Summary:Linux Twitter client designed for multiple columns > > of multiple accounts > Linux, eh? ;-) Ok ok, I removed it ;) > > desktop-file-install \ > > … > > desktop-file-validate > > Note that desktop-file-install also validates the file, so you only need to > run either one, not both. Ack. Removed. > > %{_datadir}/icons/*/*/*/* > > If the package doesn't "Requires: hicolor-icon-theme", there are multiple > unowned directories in the /usr/share/icons/ tree. > > > https://fedoraproject.org/wiki/Packaging: > Guidelines#File_and_Directory_Ownership > https://fedoraproject.org/wiki/Packaging:UnownedDirectories Ah wasn't aware of this. Fixed. > > %{_datadir}/%{name}/iconsets > > Here the directory %{_datadir}/%{name} is not included. Fixed as well. Hohum fedora-review barfs a bit due to the spaces+() in the tarname and the directory. I'll look into that and then upload a fresh package. Thanks for the feedback and regards, Michele
Polly developer here. Thank you very much for your effort! I don't know anything about Fedora packaging, but if any of the issues found is something better solved upstream, please let me know.
Hi Marcelo, actually I noticed in the latest upstream tgz you have added a COPYRIGHT file that states the following: Files: share/icons/* Copyright: 2011, Joern Konopka 2011, Georgi Karavasilev License: CC-BY-NC-SA-3 This work is licensed under a Creative Commons Attribution-NonCommercial-ShareAlike 3.0 Unported License. You should have received a copy of the license along with this work. If not, see <http://creativecommons.org/licenses/by-nc-sa/3.0/>. $ find share/icons/ -type f |wc -l 25 If CC has the NC clause it won't be fit for Fedora (and neither Debian FWIW) as it makes the whole distribution/download quite a dodgy ground (even covering CD download bandwidth costs with advertising could be violating the license in certain jurisdictions). As we're really just talking about a single icon (in different sizes or themes), can you either: A) Ask the upstream authors of the icons to relicense the icons to a more friendly license (CC-BY or CC-BY-SA for example): https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing#Good_Licenses_2 B) Get a completely different icon for the program that is licensed under one of the above licenses. I theoretically could prepare a source tarball without the problematic icons (http://fedoraproject.org/wiki/Packaging:SourceURL#When_Upstream_uses_Prohibited_Code), but it makes zero sense for something that can be trivially fixed by upstream and you might want to fix it anyway if you are interested in polly being adopted by most distros ;) Thanks, Michele
I am contacting the icon authors and will post here as I make progress. They will probably agree to relicense.
*** Bug 824218 has been marked as a duplicate of this bug. ***
For the record, I eliminated the python-keyring dependency in 0.93.11. However, the icon author has disappeared from the face of the Earth. I'm still trying to contact him, but it's not the first time this happens and I have no prediction of when I'll manage to do it.
I am closing this one as I think between birdie and corebird we have already quite some decent UI-friendly clients in Fedora, this was not the case when I started packaging polly. If someone feels like pursuing this further, packages will stay up here: http://acksyn.org/files/rpms/polly/