Bug 226904 - Review Request: qmpdclient - MPD client
Summary: Review Request: qmpdclient - MPD client
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-DEADREVIEW
TreeView+ depends on / blocked
 
Reported: 2007-02-01 22:35 UTC by Michał Bentkowski
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-07-09 14:45:25 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Michał Bentkowski 2007-02-01 22:35:37 UTC
Spec URL: http://ecik.nonlogic.org/rpm/qmpdclient/qmpdclient.spec
SRPM URL: http://ecik.nonlogic.org/rpm/qmpdclient/qmpdclient-1.0.7-1.src.rpm
Description: A very advanced MPD client written in Qt4.

Comment 1 Trond Danielsen 2007-04-05 07:35:02 UTC
Are these warnings a problem?

[trondd@localhost i386]$ rpmlint -v qmpdclient-*
I: qmpdclient checking
I: qmpdclient-debuginfo checking
W: qmpdclient-debuginfo hidden-file-or-dir /usr/src/debug/qmpdclient-1.0.7/.res
W: qmpdclient-debuginfo hidden-file-or-dir /usr/src/debug/qmpdclient-1.0.7/.res
W: qmpdclient-debuginfo hidden-file-or-dir /usr/src/debug/qmpdclient-1.0.7/.ui
W: qmpdclient-debuginfo hidden-file-or-dir /usr/src/debug/qmpdclient-1.0.7/.ui
W: qmpdclient-debuginfo hidden-file-or-dir /usr/src/debug/qmpdclient-1.0.7/.moc
W: qmpdclient-debuginfo hidden-file-or-dir /usr/src/debug/qmpdclient-1.0.7/.moc

Comment 2 Michał Bentkowski 2007-04-05 09:20:08 UTC
(In reply to comment #1)
> Are these warnings a problem?
> 

I don't think so. They all are connected with debuginfo subpackage and I have no
influence on it

Comment 3 Bernard Johnson 2007-04-11 23:08:02 UTC
(In reply to comment #2)
> (In reply to comment #1)
> > Are these warnings a problem?
> > 
> 
> I don't think so. They all are connected with debuginfo subpackage and I have no
> influence on it

Install the -debuginfo package and take a look at the files in that directory. 
I'm pretty sure you'll find that it's preprocessed code from the compile, which
I'm assuming that you'll need for the -debuginfo packages.

Comment 4 Jason Tibbitts 2007-05-31 05:14:37 UTC
I've no idea why the Component changed like that.

Comment 5 Jason Tibbitts 2007-05-31 22:35:34 UTC
I don't see any problems with the rpmlint warnings.  find-debuginfo.sh has
decided they are source and need to be in the -debuginfo subpackage, and I
certainly wouldn't want to remove them.

Any chance you could perhaps flesh out the summary a bit and maybe define "MPD"
in the %description?

I would suggest something like:

Summary: Easy to use MPD client written in Qt4 
%description
A very advanced and easy to use MPD (Music Player Daemon) client written in Qt4.
 You will need a running MPD instance running somewhere on your network in order
to make use of this software.

Other than that, this package looks good.  I'll go ahead and approve it and you
can provide something more descriptive when you check in.

Review:
* source files match upstream:
   4d01940020414cb5c16299dd03f32bc00daa2defc7805348b71ef7ff42d2c5d2
   qmpdclient-1.0.7.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
X summary is OK.
X description is a bit lacking.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* debuginfo package looks complete.
* rpmlint has only ignoreable complaints.
* final provides and requires are sane:
   qmpdclient = 1.0.7-1.fc7
  =
   /bin/sh
   hicolor-icon-theme
   libQtCore.so.4()(64bit)
   libQtGui.so.4()(64bit)
   libX11.so.6()(64bit)
   libgcc_s.so.1()(64bit)
   libgcc_s.so.1(GCC_3.0)(64bit)
   libpthread.so.0()(64bit)
   libpthread.so.0(GLIBC_2.2.5)(64bit)
   libstdc++.so.6()(64bit)
   libstdc++.so.6(CXXABI_1.3)(64bit)
   libstdc++.so.6(GLIBCXX_3.4)(64bit)

* %check is not present; no test suite upstream.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* scriptlets OK (gtk-update-icon-cache).
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la droppings.
* desktop file present: looks OK and is installed properly.

Comment 6 Michał Bentkowski 2007-07-09 11:13:56 UTC
Sorry for a late answer.
Well, apart from qmpdclient I packaged sonata - another mpd client. It appeared
that sonata is much better mpd client for me so I am no longer interested in 
qmpdclient since I don't use it anymore.


Comment 7 Jason Tibbitts 2007-07-09 14:45:25 UTC
Withdrawing approval and closing this ticket.


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