Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at

Bug 213193

Summary: Review Request: gaim-rhythmbox - Rhythmbox plugin for GAIM
Product: [Fedora] Fedora Reporter: Michel Alexandre Salim <michel.salim>
Component: Package ReviewAssignee: Wart <wart>
Status: CLOSED ERRATA QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: wart
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-11-04 05:00:43 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 163779    

Description Michel Alexandre Salim 2006-10-31 04:33:59 UTC
Spec URL:
The Gaim-Rhythmbox plugin will automatically update your Gaim user and
status info with the currently playing music in Rhythmbox.

If the artist and title are known, it will also attempt to create a
link to the song's lyrics by using Google's "I'm Feeling Lucky"

Gaim-Rhythmbox will replace %rb in your user info and status message
with the song information.

Note: this is a beta version, targeting the beta release of GAIM that comes with FC6. When approved it'll be FC6+ only.

Comment 1 Jochen Schmitt 2006-10-31 20:16:15 UTC
+ Local build works fine.
+ No complaints for source rpm.
+ No complaints for Binary rpms.
+ No complaitns for install rpm.
+ Tarball in source rpm matches with upstream.
+ Package contains verbatin License.
+ Able to install and uninstall package localy.

Unfortunateley, I was not able to check out if the plugin itself work properly.

Comment 2 Michel Alexandre Salim 2006-10-31 23:26:14 UTC
No IM account to test it with?

From my (limited) testing it works perfectly with AIM (both in status and info).
With Google Talk it works, but there is an open tag <rb> that's inserted before
the song name without a close tag.

Do let me know if you want to do a full review. Thanks!

Comment 3 Wart 2006-11-02 23:58:01 UTC
You're missing a couple of build requirements:

BR: gtk2-devel dbus-glib-devel dbus-devel

I'll pick up the full review tonight if Jochen doesn't get to it earlier.

Comment 4 Michel Alexandre Salim 2006-11-03 00:58:17 UTC
Jochen's no longer CC'ed, so I guess you get to review it.

I'm creating a mock tree right now to test it further - ldd
/usr/lib64/gaim/ disturbingly claims that the .so file requires
anything from Xinerama to libxml2. Is that normal?

Comment 5 Wart 2006-11-03 02:40:26 UTC
* rpmlint output clean
* Package named appropriately
* Source matches upstream:
  a9e836986dae7857b408120782264d5a  gaim-rhythmbox-2.0beta3.tar.gz
* Builds in mock on FC6-i386, FC6-x86_64, FC7-i386, FC7-x86_64
* GPL license ok, license file included
* Spec file legible and in Am. English.
* Runs without crashing.  Seems to work as expected with my AIM account.
* No missing BR:
* No locales
* Not relocatable
* Not a gui app; no need for a .desktop file
* No need to run ldconfig; .so files are application plugins that aren't
  part of the system linker path.
* Directory ownership ok
* No duplicate %files
* No need for -doc or -devel subpackages

* Inconsistent use of the custom 'prever' macro.  You only use it once
  in %prep, but not at all in Source0 or Release.  Either use it in all
  3 places, or not at all.

* You could also include AUTHORS and README in %doc
* There's no need to split each sentence in %description into a separate
  paragraph.  It just adds unnecessary whitespace and doesn't make it any
  easier to read.
* Send the configure patch upstream so that it can be included in the final
* I wouldn't worry about the shared library dependencies in the .so file.
  If you run ldd on the gaim executable itself, you'll see an almost-identical
  list of dependencies.

Not much here.  Just fix the use of the prever macro and you're good to go.

Comment 6 Michel Alexandre Salim 2006-11-03 03:16:33 UTC

Fixed. Sorry about %description, I copied it from upstream and did not think to
reformat it.

Comment 7 Wart 2006-11-03 03:38:18 UTC
Changes look good.


Comment 8 Michel Alexandre Salim 2006-11-04 05:00:43 UTC
Thanks! Actually, we missed a Require: on gaim. I'll put it in the next time
there's an update, since it should only get triggered if someone 'yum remove'
gaim and gaim-rhythmbox does not get removed as well.

Comment 9 Wart 2006-11-04 06:21:04 UTC
The Requires: on gaim is picked up automatically:

$ rpm -q --requires gaim-rhythmbox | grep gaim
$ rpm -q --whatprovides ''

I also verified this by running 'yum remove gaim' and saw that it would remove
gaim-rhythmbox, too.

Comment 10 Michel Alexandre Salim 2006-11-04 16:08:54 UTC
Bizarre, that does not appear on my system when using mock to build the package.
When /not/ using mock the dependency appears properly.. I'll wait for the FE
result to comes out and reverify.

% rpm -q gaim-rhythmbox

% rpm -q --requires gaim-rhythmbox | grep gaim

Comment 11 Fedora Update System 2022-10-03 18:15:41 UTC
FEDORA-EPEL-2022-bc15c240ad has been submitted as an update to Fedora EPEL 9.

Comment 12 Fedora Update System 2022-10-04 02:25:53 UTC
FEDORA-EPEL-2022-bc15c240ad has been pushed to the Fedora EPEL 9 testing repository.

You can provide feedback for this update here:

See also for more information on how to test updates.

Comment 13 Fedora Update System 2022-10-11 10:53:38 UTC
FEDORA-EPEL-2022-bc15c240ad has been pushed to the Fedora EPEL 9 stable repository.
If problem still persists, please make note of it in this bug report.