[Bug 202496] Review Request: quodlibet - A music management program

Fri Aug 18 05:46:06 UTC 2006

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: quodlibet - A music management program

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=202496

------- Additional Comments From jeff at ocjtech.us  2006-08-18 01:36 EST -------
It looks like the unit tests are only in SVN:

http://svn.sacredchao.net/svn/quodlibet/trunk/quodlibet/_sanity.sh
http://svn.sacredchao.net/svn/quodlibet/trunk/quodlibet/tests/

I chatted with the developer on IRC and he said that the tests use PyGTK which
needs X so that's a non-starter for a rpm build.  He also said that the tests
use a number of other libraries that may or may not be in Fedora.  See:

http://www.sacredchao.net/quodlibet/changeset/3795

(In reply to comment #7)

> Also, what's the upstream for google.py?  From the looks of things, it's you,
> which would be OK.

Yup, it's me.

> It seems to me that there must be a simpler way to copy 50 sources into place.
> 
> The README file says there are unit tests included, but I don't see them in the
> package.
> 
> I built this for i386 and installed it on my home machine; it seems to run OK
> but I saw this odd bit on startup:
> 
> Introspect error: The name org.gajim.dbus was not provided by any .service files

Probably because you don't have gajim installed.  One of the plugins is for 

> I found errors with four plugins:
> 
> ImportError: No module named musicbrainz

Expected... libmusicbrainz is in Core, but the python-ctypes package needed for
the libmusicbrainz python bindings are in Extras.  There is now a
python-musicbrainz2 package in extras but the API is completely different from
the original bindings.

> ImportError: No module named CDDB
> ImportError: No module named lastfm

Neither of these modules are available in Fedora at the moment.  My reasoning
for including these plugins anyway is that Quod Libet handles errors like this
cleanly, and if the user was to obtain these modules some other way (say by
manually installing from the source) the plugins would work for them.

> ImportError: No module named path
> 
> I guess the first three are optional requirements, which should be OK.  The last
> one is odd; here's the full backtrace:
> 
> Traceback (most recent call last):
>   File "/usr/share/quodlibet/plugins/songsmenu/shell.2.py", line 11, in ?
>     import os
>   File "/usr/lib/python2.4/os.py", line 133, in ?
>     from os.path import (curdir, pardir, sep, pathsep, defpath, extsep, altsep,
> ImportError: No module named path
>
> In addition, I chose View->File System and got this backtrace:
> 
> Traceback (most recent call last):
>   File "/usr/share/quodlibet/qltk/quodlibet.py", line 518, in select_browser
>     self.browser = Browser(library, player)
>   File "/usr/share/quodlibet/browsers/filesystem.py", line 67, in __init__
>     dt = DirectoryTree(initial=const.HOME, folders=folders)
>   File "/usr/share/quodlibet/qltk/filesel.py", line 68, in __init__
>     folders = filter(is_folder, folders)
> UnboundLocalError: local variable 'is_folder' referenced before assignment

The developer feels that these tracebacks may indicate some local problem with
the Pythin library.  I was unable to duplicate the problem on by screen

> I exited the program which left the default view that way and now neither
> quodlibet nor exfalso will start.

You should be able to get the view back to the default by editing
~/.quodlibet/config and changing "browser = FileSystem" to "browser = SearchBar".

> For some reason I don't get any desktop menu entry, even though the desktop file
> looks OK to me.  (I use KDE.)
> 
> Review:
> * All 9000 source files match upstream:
>    195c39433b4e2a5bf930af7570b61f3e  quodlibet-0.23.tar.gz
> 
>    0b7d2ff31c4c113b52b74e1b8c867843  albumart.py
>    2385a15ea0f8c4ccbcf347eb9e5ef174  _amazon.py
>    b5f85eb37b75acba938d082913d31b32  animosd.py
>    17421585b258ac44a0c04a50f06b25da  ape2id3.py
>    680c7f7df3ac0cd949daa153c419e451  automask.py
>    f3d86669ab18d4c25075212943cd74e1  autorating.py
>    9a984fb37453ff86ac861b01b31f6594  bookmarks.py
>    3865f4f3ece8d667c20fe3b8d34418dd  brainz.py
>    ab800ac4b48a0a228f1504914f198b5a  browsefolders.py
>    ae0be895b7441c9d38b0d8725f53f46c  cddb.py
>    73c4409dfbdc8a9d8b7cd1a04f86ac23  clock.py
>    c898ef2d38ad04b64137f0b6a8845852  fixlatin1tags.py
>    0fb2b513d89026c809b77c423c5080f8  gajim_status.py
>    ae32b874d01fb16bed24626602ed2a98  html.py
>    4b71dfdbed6d5e8248028f62a3290888  iconv.py
>    cd8f349983822b0429ef36d51c506992  ifp.py
>    4fd524e514cb0da6714628c7c1078242  importexport.py
>    57c82a80c4f30f6ac9b87801e7c80728  iradiolog.py
>    d73546846d3280d0d3b2548a97bd8991  jep118.py
>    692f212693a86de9858e4979d6cec1b6  k3b.py
>    437ab5e65b7788641880b1ff7f4ae9f4  kakasi.py
>    57cf89358975fc75267bbff6d629749f  lastfmsubmit.py
>    c235836bb4b3f0480f674b6d21996407  nautilus.py
>    2a140f01260cb7dc5fc89f95fe11b4e0  notify.py
>    d9868612bc1a13536f37d321bd4748d2  qlscrobbler.py
>    77dd97961b563ce6a0e0e596d98c39fe  randomalbum.py
>    f179c4db522561eed1629854b8de3a4d  replaygain.py
>    cb8f5b018503b2f0ad44fc10fc0bd3be  reset.py
>    dbd51ebd76b5a8e0a23558480c79da7b  resub.py
>    e5fc6a506f63de6f9661d2ded274d70d  splitting.py
>    09c6da4082b6aa33d998991be7c508a8  _subprocobj.py
>    18ba6a789e95d5d202d911d826d94747  titlecase.py
>    12a36f1cc82b14cb25ba54b3d84503c8  trayicon.py
>    87065bf6d3b2084ca1a1c1f06f2b3f6c  wikipedia.py
>    705e734f3d755a1e05854bc68ed90f6a  write_cover.py
> 
>   Source58, pulled manually according to the comments, also matches:
>    5a9d7e599fab9be828a6244ee12e3542  shell.2.py?format=raw
> 
>   No upstream for google.py
> 
> * package meets naming and packaging guidelines.
> * specfile is properly named, is cleanly written and uses macros consistently.
> * dist tag is present.
> * build root is correct.
> * license field matches the actual license.
> X license is open source-compatible.  License text upstream but not 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).
> * debuginfo package looks complete.
> X rpmlint is silent (just one tab)
> * final provides and requires are sane:
>    _mmkeys.so()(64bit)
>    quodlibet = 0.23-3.fc6
>   =
>    /usr/bin/env
>    gnome-python2-libegg
>    gstreamer-python >= 0.10
>    libatk-1.0.so.0()(64bit)
>    libcairo.so.2()(64bit)
>    libgdk-x11-2.0.so.0()(64bit)
>    libgdk_pixbuf-2.0.so.0()(64bit)
>    libglib-2.0.so.0()(64bit)
>    libgmodule-2.0.so.0()(64bit)
>    libgobject-2.0.so.0()(64bit)
>    libgtk-x11-2.0.so.0()(64bit)
>    libpango-1.0.so.0()(64bit)
>    libpangocairo-1.0.so.0()(64bit)
>    pygtk2 >= 2.8
>    python >= 2.4
>    python-mutagen >= 1.5
>    vorbisgain
> ? %check is not present; no test suite upstream.  Manual testing reveals a few
> problems; see above.
> * shared libraries are present, but they aren't in the ld search path.
> * package is not relocatable.
> * owns the directories it creates.
> * doesn't own any directories it shouldn't.
> * no duplicates in %files.
> * file permissions are appropriate.
> * no scriptlets present.
> * 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 libtool .la droppings.
> * desktop file looks correct and is installed properly.  (Note that I don't get
> a desktop menu entry, but I don't see anything wrong with the desktop file.)
> 

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.