|Summary:||Review Request: qmpdclient - MPD client|
|Product:||[Fedora] Fedora||Reporter:||Michał Bentkowski <mr.ecik>|
|Component:||Package Review||Assignee:||Jason Tibbitts <tibbs>|
|Status:||CLOSED NOTABUG||QA Contact:||Fedora Package Reviews List <fedora-package-review>|
|Fixed In Version:||Doc Type:||Bug Fix|
|Doc Text:||Story Points:||---|
|Last Closed:||2007-07-09 14:45:25 UTC||Type:||---|
|oVirt Team:||---||RHEL 7.3 requirements from Atomic Host:|
|Bug Depends On:|
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.