Bug 653971 - Reviews Request: openfetion - A Fetion client written using GTK2+
Reviews Request: openfetion - A Fetion client written using GTK2+
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
Unspecified Unspecified
low Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
Depends On: 666190
  Show dependency treegraph
Reported: 2010-11-16 09:51 EST by Liang Suilong
Modified: 2013-02-19 05:47 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2013-02-19 05:47:42 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Liang Suilong 2010-11-16 09:51:49 EST
SRPM: http://liangsuilong.fedorapeople.org/openfetion-2.0.2-2.fc13.src.rpm
SPEC: http://liangsuilong.fedorapeople.org/openfetion.spec
Openfetion is a Fetion client written using GTK2+, based on Fetion v4 protocol.

Koji Result:
F14: http://koji.fedoraproject.org/koji/taskinfo?taskID=2604205
Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=2604091

On koji, it is built for F-14 target. But for Rawhide target, It does not work. I guess there is some problem of dbus-glib for Rawhide branch.
Comment 1 Michael Schwendt 2010-11-21 14:34:08 EST
A brief look at the spec file told me the following:

> Summary:		A Fetion client written using GTK2+

> %description
> Openfetion is a Fetion client written using GTK2+,
> based on Fetion v4 protocol.

Both %summary and %description could try to explain what "Fetion" is. For example:

  Summary: GTK2+ based client for the Fetion instant messaging protocol

And %description should expand on that. Just repeating a "Name" isn't helpful.

> %setup -q -n %{name}-%{version}

Option  -n %{name}-%{version}  is the default.

> %files -f %{name}.lang
> %defattr(-,root,root,-)

Several directories are not included in the package correctly:



As a directory added to a  %files  section will be included recursively, you may need to include directories only with the  %dir  attribute.

> %files -n %{libname}
> %defattr(-,root,root,-)
> %{_datadir}/%{name}/resource/city.xml
> %{_datadir}/%{name}/resource/province.xml

Same here. Plus, you need to decide on directory ownership between this %libname package and the main %name package.

> %{_libdir}/libofetion.so*

This is wrong. Only the *.so.* files belong into the %libname package. The *.so symlink belongs into the -devel package.
Comment 2 Miroslav Suchý 2012-12-16 07:45:49 EST
Ping? Any progress here? Or we can close this review?
Comment 3 Miroslav Suchý 2013-02-19 05:47:42 EST
Stalled Review. Closing per:
If you ever want to continue in this review, please reopen or
submit new review.

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