Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 173498 - libtunepimp: A library for creating MusicBrainz enabled tagging applications
Summary: libtunepimp: A library for creating MusicBrainz enabled tagging applications
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Aurelien Bompard
QA Contact: David Lawrence
URL: http://musicbrainz.org/products/tunep...
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2005-11-17 17:08 UTC by Rex Dieter
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2005-11-20 19:29:17 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)

Description Rex Dieter 2005-11-17 17:08:44 UTC 
Spec Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/SPECS/libtunepimp-0.4.0-1.spec
SRPM Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/all/SRPMS.unstable/libtunepimp-0.4.0-1.src.rpm
Description: 
The TunePimp library (also referred to as libtunepimp) is a development library geared towards developers who wish to create MusicBrainz enabled tagging applications.

%changelog
* Thu Nov 17 2005 Rex Dieter <rexdieter[AT]users.sf.net> 0.4.0-1
- 0.4.0
- built with --enable-lgpl which omits mp3 plugin bits
Comment 1 Rex Dieter 2005-11-17 17:16:10 UTC 
Library (Theoretically) usable by: kdemultimedia(extras/juk), amarok
Comment 2 Rex Dieter 2005-11-17 20:00:19 UTC 
FYI, 

amarok requires patching to use the new libtunepimp, see
http://bugs.kde.org/94988

kdemultimedia(-3.5.0) as well:
http://bugs.kde.org/116575
Comment 3 Ville Skyttä 2005-11-17 20:38:11 UTC 
Do you happen to have any info whether the current kid3 in Extras would work 
with this version?  It should have support for libtunepimp, but I've never 
tried it out.
Comment 4 Rex Dieter 2005-11-17 20:57:00 UTC 
I'll go check it out.
Comment 5 Rex Dieter 2005-11-17 21:05:37 UTC 
Looks like kid3 uses code similar to both juk and amarok, and will need to be
likewise patched (the api changed a wee bit from lintunepimp-0.3.0 to 0.4.0).
Comment 6 Aurelien Bompard 2005-11-19 10:01:15 UTC 
Missing BR: zlib-devel and libvorbis-devel.
Also, it is BR: libmusicbrainz-devel and not BR: libmusicbrainz
Comment 7 Rex Dieter 2005-11-19 18:11:46 UTC 
FYI, see also libtunepimp-mp3 submission to livna:
http://bugzilla.livna.org/show_bug.cgi?id=637
Comment 8 Rex Dieter 2005-11-19 18:18:08 UTC 
Spec Name or Url:
http://apt.kde-redhat.org/apt/kde-redhat/SPECS/libtunepimp-0.4.0-2.spec
SRPM Name or Url:
http://apt.kde-redhat.org/apt/kde-redhat/all/SRPMS.unstable/libtunepimp-0.4.0-2.src.rpm

%changelog
* Sat Nov 19 2005 Rex Dieter <rexdieter[AT]users.sf.net> 0.4.0-2
- BR: libmusicbrainz -> libmusicbrainz-devel
- BR: libogg-devel -> libvorbis-devel
- BR: zlib-devel
Comment 9 Aurelien Bompard 2005-11-20 05:41:39 UTC 
Review for release 2:
* RPM name is OK
* Source libtunepimp-0.4.0.tar.gz is the same as upstream
* This is the latest version
* Builds fine in mock
* rpmlint of libtunepimp looks OK
* rpmlint of libtunepimp-devel looks OK
* File list of libtunepimp looks OK
* File list of libtunepimp-devel looks OK
* Works fine
APPROVED

I've added the patch to amarok, will request a rebuild when this is published.
Comment 10 Rex Dieter 2005-11-20 19:29:17 UTC 
Thanks, imported.
Comment 11 Ville Skyttä 2005-11-20 20:21:03 UTC 
No component for this yet in Bugzilla, so I'll spam here: some of the headers      
in -devel contain susceptible #includes that aren't shipped in it (and -devel   
has no other -devel deps, and even if it did, these would look more or less 
strange):   
   
track.h:#include "mutex.h"  
track.h:#include "metadata.h"  
track.h:#include "filelookup.h"  
tunepimp.h:#include "mutex.h"  
tunepimp.h:#include "filecache.h"  
tunepimp.h:#include "analyzer.h"  
tunepimp.h:#include "submit.h"  
tunepimp.h:#include "lookup.h"  
tunepimp.h:#include "filelookup.h"  
tunepimp.h:#include "write.h"  
tunepimp.h:#include "readmeta.h"  
tunepimp.h:#include "plugins.h"
Comment 12 Rex Dieter 2005-11-21 14:06:33 UTC 
Ville, 
Yeah, clearly the C++ API is not finalized, and some of the included headers
don't seem 100% usable (ie, tunepimp.h).  The only header I know that
libtunepimp-using apps are actively using is tp_c.h.
Comment 13 Ville Skyttä 2005-11-21 18:27:55 UTC 
Ok, in that case I'd suggest purging the currently unusable headers from the 
-devel package (track.h, tunepimp.h unless I missed something) in order to 
avoid confusion.
Note You need to log in before you can comment on or make changes to this bug.