Bug 983774 - Review Request: xmp-plugin-audacious - A multi-format module playback plugin for Audacious
Summary: Review Request: xmp-plugin-audacious - A multi-format module playback plugin ...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michael Schwendt
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-11 23:51 UTC by Dominik 'Rathann' Mierzejewski
Modified: 2013-08-31 17:04 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-08-31 17:04:24 UTC
Type: ---
Embargoed:
bugs.michael: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Dominik 'Rathann' Mierzejewski 2013-07-11 23:51:36 UTC
Spec URL: http://rathann.fedorapeople.org/review/xmp-plugin-audacious/xmp-plugin-audacious.spec
SRPM URL: http://rathann.fedorapeople.org/review/xmp-plugin-audacious/xmp-plugin-audacious-4.0.0-0.1.8c492d7.fc19.src.rpm
Description: 
Libxmp is a library that renders module files to PCM data. It supports
over 90 mainstream and obscure module formats including Protracker (MOD),
Scream Tracker 3 (S3M), Fast Tracker II (XM), and Impulse Tracker (IT).

Many compressed module formats are supported, including popular Unix, DOS,
and Amiga file packers including gzip, bzip2, SQSH, Powerpack, etc.

This package contains the xmp plugin for the Audacious media player.

Fedora Account System Username: rathann

Comment 1 Michael Schwendt 2013-07-15 14:47:26 UTC
> Provides: audacious-plugin-xmp = %{version}-%{release}
> Obsoletes: audacious-plugin-xmp <= 3.5.0-3

The "Obsoletes" tag is not high enough. It must be

  Obsoletes: audacious-plugin-xmp < 3.5.0-4
or
  Obsoletes: audacious-plugin-xmp <= 3.5.0-3.fc20  (with hardcoded .fc20!)

because the last build of that package has been a subpackage of xmp-3.5.0-3.fc20 (also for .fc19 and .fc18).


The "Provides" shall be kept to meet the requirements of the "Addon Packages (General) Naming Guidelines": https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Addon_Packages_.28General.29

Those guidelines aren't too precise, however, since for Audacious the %{parent} name may be "audacious" as well as "audacious-plugins". There's no requirement for a separate plugin to use the audacious-plugins- prefix.

It would be an opportunity to s/plugin/plugins/ in the "Provides", however.


> License: GPLv2+

Confirmed.


> %description
> ...
> and Amiga file packers including gzip, bzip2, SQSH, Powerpack, etc.

"PowerPacker" is its name.
( http://rewiki.regengedanken.de/wiki/Amiga_PowerPacker )


> Summary: A multi-format module playback plugin for Audacious using libxmp

In package installers, which display these summaries, it looks better
(and more concise) when omitting these leading articles:

  Summary: Multi-format module playback plugin for Audacious using libxmp

https://fedoraproject.org/wiki/Examples_of_good_package_summaries



Any fixes can be applied in pkg git.

APPROVED

Comment 2 Dominik 'Rathann' Mierzejewski 2013-07-16 12:56:46 UTC
New Package SCM Request
=======================
Package Name: xmp-plugin-audacious
Short Description: Multi-format module playback plugin for Audacious using libxmp
Owners: rathann
Branches: f19
InitialCC:

Comment 3 Dominik 'Rathann' Mierzejewski 2013-07-16 12:58:23 UTC
Thanks for the quick review, Michael. I'll fix the issues you highlighted before building. Feel free to apply for co-maintainer status once this is added to PkgDB.

Comment 4 Gwyn Ciesla 2013-07-16 13:31:04 UTC
Git done (by process-git-requests).


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