| Summary: | Review Request: polly - Linux Twitter client designed for multiple columns of multiple accounts | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Michele Baldessari <michele> |
| Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
| Status: | CLOSED DEFERRED | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | rawhide | CC: | aalves, conscioususer, i, jv+fedora, nekohayo, package-review |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2014-02-25 23:14:40 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Bug Depends On: | 1007354 | ||
| Bug Blocks: | |||
|
Description
Michele Baldessari
2013-11-02 23:38:47 UTC
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/ |