Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 218256 - Review Request: audacious-itouch-control - iTouch keyboard control plugin for the Audacious media player
Summary: Review Request: audacious-itouch-control - iTouch keyboard control plugin for...
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Christoph Wickert
QA Contact: Fedora Package Reviews List
Depends On:
TreeView+ depends on / blocked
Reported: 2006-12-04 07:34 UTC by Yu Fan
Modified: 2008-06-17 19:48 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2008-05-25 20:00:18 UTC

Attachments (Terms of Use)

Description Yu Fan 2006-12-04 07:34:24 UTC
Spec URL: <>
Description: <With this Audacious plugin you can take advantage of the
multimedia (playback and volume control) keys on your Logitech iTouch or
compatible keyboard. When
the plugin is used you can use the keys regardless of the current input focus.
The plugin won't work if some other application (eg. xscreensaver) has grabbed
the keyboard.>

Comment 2 Till Maas 2007-01-17 23:08:05 UTC
Consider renaming it to audacious-plugins-itouch because this fits to the naming
scheme of the other audacious plugins in extras.

Comment 3 Christoph Wickert 2007-01-17 23:40:37 UTC
If this package should be renamed, then audacious-docklet also needs to be renamed.

If this package is going to be renamed, we still should add Provides: and
Obsoletes: for audacious-itouch to allow easy upgrading of 3rd party packages.

I don't think renaming this package is really necessary. According to the naming
guidlines an addon package "should prepend the "parent" package in its name, in
the format: %{parent}-%{child}." IMHO Using audaciuos as parent is perfectly ok.
Using audaciuos-plugins would indicate that this package stricktly depends on
audacious-plugins instead of audacious itself.
The only reason for this package to depend on audacious-plugins is that the
plugins package owns the dir this plugin is installed to.

Comment 4 Till Maas 2007-01-18 15:41:33 UTC
I suggested to rename the package not because of the guidelines but because it
is confusing for me as a user that the plugins for audacious do not have a
consistent naming scheme. I did not know that audacious-plugins-* are all from
the same package.

Comment 5 Christoph Wickert 2007-01-20 17:58:04 UTC
I still think that audacious-itouch is a valid name for this package, since this
is the upstream name. Nevertheless - after reading the review request for
audacious-plugin-fc (bug #222648) and thinking about this more deeply, I like
the idea of getting this straight. This would also include renaming

Yu, what you think?

Comment 6 Till Maas 2007-01-20 21:33:55 UTC
To me it seems best to ask the packaging committee for a recomandation. Right
now it seems that "<package>-<pluginname>", "<package>-plugin-<pluginname>",
"<package>-plugins-<pluginname>" and "<package>-<pluginname>-plugin" for
different packages. I would prefer "<package>-plugins-<pluginname>" or
"<package>-plugin-<pluginname>", this would make it easier to install all
plugins, e.g. with yum install audacious-plugins-\* while audacious-\* also
matches the devel package.

Comment 7 Yu Fan 2007-01-29 10:28:37 UTC
I agree the plugin should be named audacious-plugin-itouch.
But to do this, I must modify the source tarbal, and repackage it. Do I have the
right to do that? 
Besides of that, I have made the package available for a while. When will it get
pass the review process? 

Comment 8 Kelly Miller 2007-05-09 21:20:47 UTC
> I agree the plugin should be named audacious-plugin-itouch.
> But to do this, I must modify the source tarbal, and repackage
> it. Do I have the right to do that? 

Nah, you don't have to.  %setup accepts a switch, -n, which allows for 
changing the exact name of the folder in the source tarball.  In this case, 
simply change to this:
%setup -q -n audacious-itouch-%{version}

Quick Review (not official):
Since I can't download the source, this is just going to be a quick 
MUST Items: 
- rpmlint gives no errors.
- The package is named according to the Package Naming Guidelines.
- The package is licenced under an open-source licence (GPL).
- Licence info included in %doc.
- Spec file named %{name}.spec.
- Spec file written in en-US and is legible.
- The package compiles on i386.
- Build dependencies are listed in BuildRequires properly.
- No locales in the package.
- ldconfig isn't called, but the library is not located in the standard linker 
- Package is not relocatable.
- Package owns all directories it creates.
- Permissions on files are set properly.
- Package has a proper %clean section.
- Spec file uses consistent macros.
- Package contains permissible content.
- All files in %doc are not necessary for program application.
- .la files are properly removed.
- Package does not create it's own directories.
- Package runs %{__rm} -rf %{buildroot} at the beginning of install.
- All filenames are valid UTF-8.

Possible problems:
- As mentioned before, the package should probably be called 
- The Source entry is actually a 404 now.

Comment 9 Jason Tibbitts 2007-06-21 16:35:07 UTC
Anything happening here?  It's been well over a month since the last comment
without any response.

Comment 10 Christoph Wickert 2007-07-05 12:42:15 UTC
Yu is going to update the package and I will do the final review to get this
thing done.

Comment 11 Jason Tibbitts 2008-01-18 05:39:33 UTC
Well, five more months have passed.  It's not as if this is time-sensitive, but
it would be nice to get some of these old submissions reviewed and closed out.

Comment 12 Christoph Wickert 2008-04-05 14:00:29 UTC
Yu, unfortunately this project is dead upstream, so I suggest we close this
review. Not even sure if it is still compatible with the current audacious API.

If you are still interesting in maintaining an audacious package you could do
audacious-show from There
already was a review for it but the reporter has withdrawn the request, see bug

Setting this bug to NEEDINFO. If I don't hear anything from you within the next
2 weeks I'll close this review.

Comment 13 Brian Pepple 2008-05-25 20:00:18 UTC
There's been no reply to this stalled review (comment #12).  Closing bug.

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