Bug 461425 - Review Request: daap-sharp - DAAP client library for Mono
Review Request: daap-sharp - DAAP client library for Mono
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Michel Alexandre Salim
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2008-09-07 16:32 EDT by Ian Burrell
Modified: 2009-03-24 15:23 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2009-03-24 15:13:10 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Ian Burrell 2008-09-07 16:32:12 EDT
This is a re-review of an orphaned package.

Spec URL: http://znark.com/fedora/daap-sharp.spec
SRPM URL: http://znark.com/fedora/daap-sharp-0.3.5-2.fc10.src.rpm
Description: daap-sharp is a DAAP (Digial Audio Access Protocol) implementation.
It is used by Apple's iTunes software to share music.
Comment 1 Peter Lemenkov 2008-09-08 01:06:04 EDT
Seems that it's already in Fedora

Comment 2 Ian Burrell 2008-09-10 16:59:43 EDT
(In reply to comment #1)
> Seems that it's already in Fedora
> http://cvs.fedoraproject.org/viewvc/rpms/daap-sharp/

It was in Fedora.  It got orphaned and removed around Fedora 7.  It needs a re-review to be restored.
Comment 3 Michel Alexandre Salim 2008-09-22 13:12:19 EDT
Straightforward review, since the package has already been approved, so these are just clean-up requests.

daap-sharp.src: W: mixed-use-of-spaces-and-tabs (spaces: line 3, tab: line 10)

Easiest fix is to open the file in Emacs, select all, then M-x untabify

Package builds OK, but on my system that has monodoc installed, it generates some files that are currently unpackaged:

RPM build errors:
    Installed (but unpackaged) file(s) found:

These won't be generated by Koji since monodoc is not installed there, but would you consider adding a BR and packaging them, perhaps in a -doc subpackage?
Comment 4 Michel Alexandre Salim 2008-09-22 14:04:07 EDT
Fails on ppc64, not sure if it's transient or not, but you might want to take a look. If it's replicable you might want to ExcludeArch ppc64 and file a tracking bug (or reuse the Mono ppc64 tracking bug)

Comment 5 Michel Alexandre Salim 2008-10-10 09:20:39 EDT
Comment 6 Michel Alexandre Salim 2009-03-24 15:13:10 EDT
Closing due to lack of response

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