Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=430366
Summary: Review Request: <Aqualung> - <media player with native jack support ans ladspa support> Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: michael.kyne-phillips@sky.com QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
Spec URL: http://www.mediafire.com/?5tol9bb1pdt SRPM URL: http://www.mediafire.com/?cdnge2tmsan Description: First Package Need Sponsor
Aqualung is a music player for the GNU/Linux operating system.
It plays audio files from your filesystem and has the feature of inserting no gaps between adjacent tracks. It also supports high quality sample rate conversion between the file and the output device, when necessary.
Almost all sample-based, uncompressed formats (eg. WAV, AIFF, AU etc.) are supported. Files encoded with FLAC (the Free Lossless Audio Codec), Ogg Vorbis, MPEG Audio (including the infamous MP3 format) and MOD files (MOD, S3M, XM, IT, etc.) are also supported.
The program can play the music through OSS, ALSA or the JACK Audio Connection Kit.
Aqualung supports the LADSPA 1.1 plugin standard. You can use any suitable plugin to enhance the music you are listening to.
Other features of the program: internally working volume and balance controls (not touching the soundcard mixer), multiple skin support, random seeking during playback, track repeat, list repeat and shuffle mode (besides normal playback). It will come up in the same state as it was when you closed it, including playback modes, volume & balance settings, currently processing LADSPA plugins, window sizes, positions & visibility, and other miscellaneous options.
You can control any running instance of the program remotely from the command line (start, stop, pause etc.). Remote loading or enqueueing soundfiles as well as complete playlists is also supported.
In addition to all this, Aqualung provides a so-called Music Store that is an XML-based music database, capable of storing various metadata about music on your computer (including, but not limited to, the names of artists, and the titles of records and tracks). You can (and should) organize your music into a tree of Artists/Records/Tracks, thereby making life easier than with the all-in-one Winamp/XMMS playlist.
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: <Aqualung> - <media player with native jack support ans ladspa support>
https://bugzilla.redhat.com/show_bug.cgi?id=430366
michael.kyne-phillips@sky.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |177841 nThis| | Summary|Review Request: <Aqualung> -|Review Request: <Aqualung> - |<media player with native |<media player with native |jack support ans ladspa |jack support ans ladspa |support> |support>
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: Aqualung - media player with native jack support ans ladspa support
https://bugzilla.redhat.com/show_bug.cgi?id=430366
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: <Aqualung> -|Review Request: Aqualung - |<media player with native |media player with native |jack support ans ladspa |jack support ans ladspa |support> |support
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: Aqualung - media player with native jack support ans ladspa support
https://bugzilla.redhat.com/show_bug.cgi?id=430366
------- Additional Comments From simon@sxw.org.uk 2008-01-27 09:56 EST ------- Some pre-review comments - due to the issues with the spec file noted below, I haven't tried to build this package
*) The version should be the complete upstream version (that is 0.9beta9), Release should be purely the version of the downstream packaging.
*) Your Source path should include the full URL from which the upstream source can be downloaded, not just a filename (see the Tags section of the Packaging Guidelines [1], and the linked document which gives examples for the sourceforge.net case)
*) From the source code, your license should be GPLv2+, rather than GPLv2 (the source code specifically states 'or any later version) - see the Licensing Guidlines[2] for more details
*) You don't need to list gcc-c++ in your BuildRequires section - see the Packaging Guidelines for full details of packages which don't need to be listed in BuildRequires
*) I don't think you need as comprehensive a Requires section. RPM's built in dependency generator generally does a pretty good job - you should only need to list in Requires the packages that you do depend upon that RPM doesn't notice itself.
*) The rm command which cleans the buildroot should be in the %install section (not immediately before it)
*) You shouldn't use %makeinstall (see the packaging guidelines). Do you really need to do a make install, followed by %makeinstall anyway?
*) IMO, it would be neater if the .desktop file was distributed as an additional Source, rather than embedded within the spec file
*) You should be consistent with your macro use - either use %{buildroot} or $RPM_BUILD_ROOT - don't chop and change throughout the file.
*) If you're using desktop-file-install, you must include desktop-file-utils in your BuildRequires
*) The <vendor_id> in your call to desktop_file_install should be replaced with the name of the vendor (in this case, I would imagine this is fedora, as upstream aren't providing the .desktop file)
And finally:
*) The package doesn't build under Fedora 9, because of the dependency on libmad ERROR: Bad build req: No Package Found for libmad-devel. Exiting.
See the Forbidden Items list [3] for details of why Fedora can't ship packages which provide MP3 support.
[1] - http://fedoraproject.org/wiki/Packaging/Guidelines [2] - http://fedoraproject.org/wiki/Licensing [3] - http://fedoraproject.org/wiki/ForbiddenItems
Hope that's of some use ...
Simon.
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: Aqualung - media player with native jack support ans ladspa support
https://bugzilla.redhat.com/show_bug.cgi?id=430366
------- Additional Comments From michael.kyne-phillips@sky.com 2008-01-28 16:37 EST ------- Second Attempt removed non-open deps, cleaned spec and corrected .desktop.
new urls Srcrpm http://www.mediafire.com/?6nmgusydnyz Spec http://www.mediafire.com/?amm9t5ktmam
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: Aqualung - media player with native jack support ans ladspa support
https://bugzilla.redhat.com/show_bug.cgi?id=430366
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-01-30 13:09 EST ------- ffmpeg is not available on Fedora. Perhaps do you want to import this on livna?
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: Aqualung - media player with native jack support ans ladspa support
https://bugzilla.redhat.com/show_bug.cgi?id=430366
------- Additional Comments From guidolinfrancesco@email.it 2008-02-02 16:00 EST ------- (This is an informal review)
Version: 0.9beta9 Non numeric version string like alpha, beta, rc, cannot be simply putted into the version tag, you should use the Release field for this. http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-d97a3f40b6dd9d...
I think you should disable all the optional package that doesn't fill the Fedora guidelines, like ffmpeg, using the appropriate ./configure flags:
--with-oss=yes,no compile with OSS support (default: yes) --with-alsa=yes,no compile with ALSA support (default: yes) --with-jack=yes,no compile with JACK support (default: yes) --with-src=yes,no compile with Sample Rate Converter support (default: yes) --with-sndfile=yes,no compile with sndfile (WAV, AIFF, etc.) support (default: yes) --with-flac=yes,no compile with FLAC support (default: yes) --with-ogg=yes,no compile with Ogg Vorbis support (default: yes) --with-vorbisenc=yes,no compile with Ogg Vorbis encoding support (default: yes) --with-speex=yes,no compile with Ogg Speex support (default: yes) --with-mpeg=yes,no compile with MPEG Audio support (default: yes) --with-mod=yes,no compile with MOD Audio support (default: yes) --with-mpc=yes,no compile with Musepack support (default: yes) --with-mac=yes,no compile with Monkey's Audio Codec support (default: yes) --with-lavc=yes,no compile with lavc (FFmpeg) support (default: yes) --with-lame=yes,no compile with LAME (MP3 encoding) support (default: yes) --with-wavpack=yes,no compile with WavPack support (default: yes) --with-ladspa=yes,no compile with LADSPA plugin support (default: yes) --with-cdda=yes,no compile with CDDA support (default: yes) --with-cddb=yes,no compile with CDDB support (default: yes) --with-ifp=yes,no compile with iRiver iFP driver support (default: yes) --with-systray=yes,no compile with Systray support (default: yes) --with-loop=yes,no compile with loop playback support (default: yes) --with-podcast=yes,no compile with podcast support (default: yes)
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: Aqualung - media player with native jack support ans ladspa support
https://bugzilla.redhat.com/show_bug.cgi?id=430366
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Flag| |needinfo?(michael.kyne- | |phillips@sky.com)
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-02-08 10:12 EST ------- ping? Unless we resolve ffmpeg problem, this review request must be dropped.
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: Aqualung - media player with native jack support ans ladspa support
https://bugzilla.redhat.com/show_bug.cgi?id=430366
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-02-16 12:58 EST ------- ping again?
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: Aqualung - media player with native jack support ans ladspa support
https://bugzilla.redhat.com/show_bug.cgi?id=430366
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-02-24 09:52 EST ------- Again ping?
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: Aqualung - media player with native jack support ans ladspa support
https://bugzilla.redhat.com/show_bug.cgi?id=430366
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|177841 |201449 nThis| | Status|NEEDINFO |CLOSED Resolution| |NOTABUG Flag|needinfo?(michael.kyne- | |phillips@sky.com) |
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-03-08 11:02 EST ------- Once closing.
If someone wants to import this package into Fedora, please file a new review request and mark this bug a duplicate of the new one. Thank you.
package-review@lists.fedoraproject.org