Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=666190 --- Comment #3 from Liang Suilong <liangsuil...@gmail.com> 2010-12-30 09:35:47 EST --- (In reply to comment #2) > Same issues as in bug 653971 (OpenFetion review): > > * Unowned directories: > > %{_datadir}/openfetion/ > %{_datadir}/openfetion/resource/ > > http://fedoraproject.org/wiki/Packaging:UnownedDirectories > I have suggested move city.xml and province.xml from %{_datadir}/openfetion/resource/ to %{_datadir}/libofetion/resource/. Openfetion and libofetion will not own the same directory. And I will fix the issues in bug 653971. > > > Summary: Library files of Openfetion > > Rather terse and unclear, IMO. It's likely that a lib* package contains > library > files, and obviously somebody who would search for "fetion" will find > libofetion. Is this library attractive to other people, too? What parts of the > Fetion protocol does it implement? E.g. > > | Summary: Instant messaging client library for the Fetion protocol V4 > > > > %description > > Fetion is an instant messaging (IM) client developed > > by China Mobile, [...] > I have rewrote a new summary and description. You can read it in next version. > But this is libofetion for OpenFetion, so the description ought to start with > explaining what _this_ package contains. A few sentences later it could > comment > on the relationship with the original Fetion or the Fetion Protocol V4. > > > > %setup -q -n %{name}-%{version} > > Option -n %{name}-%{version} is the default. No need to add it. OK. I fix it. I am waiting author's update. He says that he will push a new version. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review