Bug 469080

Summary: Review Request: pidgin-musictracker - plugin for Pidgin which displays the music track currently playing in the status message
Product: [Fedora] Fedora Reporter: Jon Hermansen <jon.hermansen>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, mail, notting, susi.lehtola
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
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: ---
Bug Depends On:    
Bug Blocks: 201449    

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.