Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 251947 - Review Request: libkarma - Library to manage the Rio Karma via TCP/IP
Summary: Review Request: libkarma - Library to manage the Rio Karma via TCP/IP
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2007-08-13 16:27 UTC by Aurelien Bompard
Modified: 2008-09-08 14:20 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2008-09-08 14:20:48 UTC

Attachments (Terms of Use)

Description Aurelien Bompard 2007-08-13 16:27:31 UTC
Spec URL:
Libkarma is a C library for managing the Rio Karma via TCP/IP. It supports all
basic protocol commands and some advanced functions on top of these, like
caching of the Rio database.

Warning: I don't have a Rio Karma to test this package, I packaged it as an extension Amarok can take advantage of. The reviewer (or one of the commenters) should test it on an actual Rio Karma device.

Comment 1 Eric Munson 2008-01-17 20:43:37 UTC
Your spec file lists two files that your install process does not seem to
create.  RPM complains about these two:
RPM build errors:
    File not found: /var/tmp/libkarma-0.1.0-1-root-/usr/lib/karma-sharp
    File not found: /var/tmp/libkarma-0.1.0-1-root-/usr/lib/pkgconfig/karma-sharp.pc

When I run the make install by hand these files are not installed.

Comment 2 Aurelien Bompard 2008-01-19 12:31:31 UTC
I'm surprised, it rebuilt fine in mock here (on an F-8 chroot). What Fedora
version did you try it on ?

Comment 3 Mamoru TASAKA 2008-01-19 13:11:07 UTC
Actually this doesn't build on dist-f9, at least on x86_64 and ppc64,
but rebuilds on at least on i386.

From build.log perhaps this is due to incorrent %_libdir <-> /usr/lib{,64}

Comment 4 Mamoru TASAKA 2008-01-19 13:14:24 UTC
You can try to rebuild your arbitrary srpm on koji by:
$ koji build --scratch <target> <your_srpm>

Currently <target> can be dist-f9-gcc43 (for gcc43 check), dist-f9,
dist-f8-updates-candidate and dist-fc7-updates-candidate.
If rebuilds is successful, 3 logs and the result rpms are put on<your_FAS_name>/task_<task_id>/ .

Comment 5 Aurelien Bompard 2008-01-26 10:45:21 UTC
OK, this one builds fine :

Comment 6 Mamoru TASAKA 2008-01-27 16:09:13 UTC

From build.log on :
it says:
    68  cc -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
-fstack-protector --param=ssp-buffer-size=4 -m32 -march=i386 -mt
une=generic -fasynchronous-unwind-tables -fPIC -D_REENTRANT -Wall -pedantic
-ggdb -W -Wchar-subscripts -Wmissing-prototypes  -Wmiss
ing-declarations -Wno-switch -Wredundant-decls -Wno-unused   -c -o mp3.o mp3.c
and src/mp3.c seems to be related with MP3 compression:

Once setting FE-Legal.

Comment 7 Jason Tibbitts 2008-01-27 21:05:21 UTC
It seems to me that mp3.c contains code to parse information out of an mp3 file
like the tags, bitrate, etc. but does not include any patented algorithms.  I
don't see anything related to audio compression, although perhaps I'm missing
something.  This package would have no reason to decode the audio from an mp3
file anyway as it's the Karma which does the playback.

These are all of the function signatures from mp3.h:
  int get_mp3_info(mp3info *mp3);
  int header_layer(mp3header *h);
  int header_bitrate(mp3header *h);
  int header_frequency(mp3header *h);
  int write_tag(mp3info *mp3);

Comment 8 Tom "spot" Callaway 2008-04-01 15:20:13 UTC
This is not mp3 playback, encode or decode. It's just scraping the ID3 tag and
mp3 metadata, presumably to make it available in some sort of client.

Lifting FE-Legal.

Comment 9 Jason Tibbitts 2008-07-02 21:04:58 UTC
This seems to have completely dropped through the cracks; is there still
interest in getting this into the distro?

rpmlint says:
  karma-sharp.x86_64: W: devel-file-in-non-devel-package 
This must be in a -devel package according to, although I don't really see what
would be hurt by just putting it in the main -devel package.

  karma-sharp.x86_64: E: only-non-binary-in-usr-lib
This is bogus.  It doesn't like just seeing a directory there.

  libkarma-devel.x86_64: W: no-documentation
Not a problem.

It looks to me like the license is GPLv2+; all of the source files seem to say
"or later".

Otherwise I think this package looks OK, although I haven't done a full review.

Comment 10 Jason Tibbitts 2008-08-21 12:18:55 UTC

I'll close this soon if there's no further response.

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