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/bugzilla/show_bug.cgi?id=215165
Summary: Review Request: audacious-plugins - Plugins for the Audacious media player Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: redhat-bugzilla@camperquake.de QAContact: fedora-package-review@redhat.com
Spec URL: http://www.skytale.net/files/audacious/audacious-plugins.spec SRPM URL: http://www.skytale.net/files/audacious/audacious-plugins-1.2.2-0.5.sky.src.r... Description:
Audacious is a media player that currently uses a skinned user interface based on Winamp 2.x skins. It is based on ("forked off") BMP. This package provides essential plugins for audio input, audio output and visualization.
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
redhat-bugzilla@camperquake.de changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |214818
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |panemade@gmail.com OtherBugsDependingO|163776 |163778 nThis| |
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From redhat-bugzilla@camperquake.de 2006-11-14 15:33 EST ------- New version to fix missing build requires at http://www.skytale.net/files/audacious/
Needs a newer version of audacious as well, available at the same place.
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From panemade@gmail.com 2006-11-15 23:43 EST ------- I dont think you need Obsoletes tag for subpackages -arts, -jack, -esd. Because they are throwing a rpmlint error.
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From redhat-bugzilla@camperquake.de 2006-11-16 03:42 EST ------- The obsoletes are necessary, because the old audacious package provided these extra packages, too, and they need to be replaced on upgrade.
Do you refer to the message that the subpackages have Obsoletes:, but no Provides:?
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From panemade@gmail.com 2006-11-16 04:07 EST ------- (In reply to comment #3)
The obsoletes are necessary, because the old audacious package provided these extra packages, too, and they need to be replaced on upgrade.
ok
Do you refer to the message that the subpackages have Obsoletes:, but no
Provides:?
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From redhat-bugzilla@camperquake.de 2006-11-16 04:26 EST ------- I am not sure about this one.
For one, there ought to be no packages which Require: audacious-arts, for example, so there is no gain in the new packages Providing: it.
In addition, if there were any other packages which actually Require: one of the Obsoleted: packages, they probably need to be rebuilt to match the new codebase, anyway, so there ought to be a RPM conflict.
So, in conclusion, I think that this specific rpmlint warning can be ignored. I'd be happy about a third opinion, though.
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
rpm@greysector.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |rpm@greysector.net
------- Additional Comments From rpm@greysector.net 2006-11-16 21:41 EST ------- (In reply to comment #5)
So, in conclusion, I think that this specific rpmlint warning can be ignored. I'd be happy about a third opinion, though.
I think it can be ignored, too.
I have another comment, however. I know it's a matter of preference, but why not put each BuildRequired package in its own line and sort the list alphabetically? It makes it easier to spot missing builddeps, for example: fluidsynth-devel and libglade2-devel. The package doesn't build here without the latter.
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
matthias@rpmforge.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |matthias@rpmforge.net
------- Additional Comments From matthias@rpmforge.net 2006-11-17 05:20 EST ------- Yeah, this also seems correct to me : No need to "provide" the old names, as they shouldn't have been used other than explicitly (not from other package requirements). The obsoletes are required though, to provide a clean upgrade path, and are correct with the last known version.
Please start the review ASAP or let me know if you'd like me to do it instead, as I'm quite impatient to have this audacious update available ;-)
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From panemade@gmail.com 2006-11-17 06:09 EST ------- Ok. Based on you comment i am going to review this package now.
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From panemade@gmail.com 2006-11-17 06:41 EST ------- (In reply to comment #7)
Yeah, this also seems correct to me : No need to "provide" the old names, as they shouldn't have been used other than explicitly (not from other package requirements). The obsoletes are required though, to provide a clean upgrade path, and are correct with the last known version.
Please start the review ASAP or let me know if you'd like me to do it instead, as I'm quite impatient to have this audacious update available ;-)
Thanks for testing this package. Based on above comment Review + package builds in mock (development i386)FC7. + rpmlint is silent for SRPM. + rpmlint on RPMs is not silent but as per above comments in bugzilla they can be ignored. + source files match upstream. 8ac7f73da7432e1ffed6c2b9b0fced8c audacious-plugins-fedora-1.2.2.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. License text included in package. + %doc is small; no -doc subpackage required. + %doc does not affect runtime. + COPYING included in %doc. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc files. + no -devel subpackage exists + available subpackages are audacious-plugins-jack,audacious-plugins-arts, audacious-plugins-esd,audacious-plugins-pulseaudio + as subpackages are packaging .so files post and postun called /sbin/ldconfig + Used update-desktop-database correctly + no .la files. + no translations available - Does NOT owns the directories it creates. + no duplicates in %files. + file permissions are appropriate.
SHOULD:- I saw that some directories are owned in audacious rpm whereas there is no need to own them by audacious but it should be owned by audacious-plugins. like /usr/lib/audacious /usr/lib/audacious/Container /usr/lib/audacious/Effect /usr/lib/audacious/General /usr/lib/audacious/Input /usr/lib/audacious/Output /usr/lib/audacious/Visualization Make it own by audacious-plugins
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From panemade@gmail.com 2006-11-17 06:56 EST ------- Matthias Saou, What do you think is above question is valid or should i approve package as it is?
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From redhat-bugzilla@camperquake.de 2006-11-17 07:07 EST ------- OK, one might argue that a-p should own /usr/lib/audacious, since audacious itself does not place any files there.
If (and that's a big if) we/I should ever decide to split up a-p into a lot of tiny subpackages these directories would have to go back to audacious, since no subpackage could own all of these dirs.
I personally do not care too much about it, but given the choice would leave things as they are (reduces work for me :)
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From matthias@rpmforge.net 2006-11-17 07:25 EST ------- Since we have audacious requiring audacious-plugins, and audacious-plugins requiring audacious, I'm not quite sure what the proper solution would be. A while back, I would have made those dirs owned by both packages, in order to be 100% sure they get rmdir'ed upon removal of both packages (since we don't know which will be removed last in the same rpm transaction), but I'm not sure this is compatible with today's packaging guidelines. You might want to ask on the packaging list or go through the guidelines 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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From redhat-bugzilla@camperquake.de 2006-11-17 07:31 EST ------- Erm, audacious Requires: audacious-plugins. audacious-plugins Requires: libaudacious.so.4 (provided by audacious-libs) audacious-plugins does not require audacious (circular deps, ugly).
So there is nothing that prevents audacious from being removed while a-p is still installed as far as I can see.
Ugly, that. Make audacious-libs own the plugin-dirs?
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From matthias@rpmforge.net 2006-11-17 08:16 EST ------- I just looked again at audacious.spec :
Requires: audacious-plugins >= 1.2.0
And at audacious-plugins.spec :
Requires: audacious >= 1.2.0
But maybe those two spec files aren't the latest?
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From redhat-bugzilla@camperquake.de 2006-11-17 09:05 EST ------- You're right, I forgot that (splitting off audacious-libs solved the build time deps, however).
But even the explicit dependency does not guarantee removal in the right order, or does it?
Any bright ideas on that?
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From panemade@gmail.com 2006-11-19 23:54 EST ------- Ralf, Got any working solution?may be then you can do make a-p own audacious-libs. Just a thought.
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From redhat-bugzilla@camperquake.de 2006-11-20 03:19 EST ------- I think I will make a-p own /usr/lib/audacious. This does not solve the question which package is to be removed first (audacious or a-p), but at least the directory structure will be cleanly installable and removable.
Modified packages forthcoming.
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From redhat-bugzilla@camperquake.de 2006-11-21 04:54 EST ------- New audacious and audacious-plugins packages at http://www.skytale.net/files/audacious/ Moves /usr/lib/audacious into audacious-plugins, and adds a patch for cdaudio.
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From panemade@gmail.com 2006-11-21 05:24 EST ------- Unable to download files
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From panemade@gmail.com 2006-11-21 07:21 EST ------- Ok got the access. I think packaging is OK now. APPROVED from me but do anyone have still any objections?
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From rpm@greysector.net 2006-11-21 08:13 EST ------- fluidsynth alsa-midi plugin is still missing, see comment #6.
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From redhat-bugzilla@camperquake.de 2006-11-21 12:46 EST ------- fluidsynth added
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From panemade@gmail.com 2006-11-21 23:00 EST ------- where? why are you not posting updated direct download links here?
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From redhat-bugzilla@camperquake.de 2006-11-22 13:42 EST ------- It's at the same place all the other versions were, posted several times though this review. I thought that was kind of evident.
http://www.skytale.net/files/audacious/audacious-plugins-1.2.2-0.8.sky.src.r... http://www.skytale.net/files/audacious/audacious-plugins.spec
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163778 |163779 nThis| |
------- Additional Comments From panemade@gmail.com 2006-11-22 23:18 EST ------- Thanks APPROVED.
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
------- Additional Comments From redhat-bugzilla@camperquake.de 2006-11-24 07:02 EST ------- Thanks.
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
redhat-bugzilla@camperquake.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
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: audacious-plugins - Plugins for the Audacious media player
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215165
Bug 215165 depends on bug 214818, which changed state.
Bug 214818 Summary: Review Request: audacious - A GTK2 based media player similar to xmms https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214818
What |Old Value |New Value ---------------------------------------------------------------------------- Resolution| |NEXTRELEASE Status|ASSIGNED |CLOSED
package-review@lists.fedoraproject.org