Bug 469080 - Review Request: pidgin-musictracker - plugin for Pidgin which displays the music track currently playing in the status message
Review Request: pidgin-musictracker - plugin for Pidgin which displays the mu...
Status: CLOSED NOTABUG
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
:
Depends On:
Blocks: FE-DEADREVIEW
  Show dependency treegraph
 
Reported: 2008-10-29 15:30 EDT by Jon Hermansen
Modified: 2009-05-22 13:04 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-12-17 03:22:28 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Jon Hermansen 2008-10-29 15:30:58 EDT
Spec URL: http://crowdsurfoffacliff.com/work/pidgin-musictracker.spec
SRPM URL: http://crowdsurfoffacliff.com/work/pidgin-musictracker-0.4.1-1.fc10.src.rpm
Description: Musictracker is a plugin for Pidgin which displays the music
track currently playing in your status.

Currently supported players: Amarok, Rhythmbox, Audacious, XMMS,
MPC/MPD, Exaile, Banshee, Quod Libet.
Comment 1 Jon Hermansen 2008-10-29 15:33:55 EDT
Sorry, I neglected to mention that this is my first package, and also that I need a sponsor.
Comment 2 Fabian Affolter 2008-11-03 06:29:51 EST
Just some small comments on your spec file

The current version is 0.4.11, perhaps you should update your spec file to reflect the latest upstream version.

- Your Source0 is not working...it should be something like 
  http://%{name}.googlecode.com/files/%{name}-%{version}.tar.bz2

- In your %prep section: %setup -q is enough, because the upstream name is now pidgin-musictracker and no longer musictracker like in the version you packed.

- You can use " find %{buildroot} -name '*.la' -exec rm -f {} ';' " in the %install section to remove the *.la. This way to don't need to exclude those files in the %files section.
Comment 3 Fabian Affolter 2008-11-13 04:48:12 EST
To find a sponsor it helps do do some informal reviews and submit some more packages.
Comment 4 Fabian Affolter 2008-11-19 06:01:44 EST
If you are looking for doing a review, check out https://bugzilla.redhat.com/show_bug.cgi?id=471575
Comment 5 Fabian Affolter 2008-11-26 04:51:11 EST
ping?
Comment 6 Fabian Affolter 2008-12-10 11:56:13 EST
It seems that you are no longer interested to bring this package into Fedora.  Is that right?

I'm asking because there is a new review request for this package.

https://bugzilla.redhat.com/show_bug.cgi?id=475799
Comment 7 Fabian Affolter 2008-12-17 03:22:28 EST
I'm close this Review request because there is an other one and the reporter seems to be not longer interested.

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