Bug 183291 - Review Request: ipod-sharp
Summary: Review Request: ipod-sharp
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Brian Pepple
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On: 183290
Blocks: FE-ACCEPT 183292
TreeView+ depends on / blocked
 
Reported: 2006-02-27 23:03 UTC by Christopher Aillon
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2006-03-10 19:57:25 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Christopher Aillon 2006-02-27 23:03:52 UTC
Spec and SRPM from: http://people.redhat.com/caillon/RPMS/rawhide/banshee/
Description: ipod-sharp provides high-level feature support for Apple's iPod and binds libipoddevice. ipod-sharp is written in C# under Mono.

Comment 1 Brian Pepple 2006-03-02 15:31:02 UTC
MD5Sums:
734956f3505f776c6920d4ae687f4bee  ipod-sharp-0.5.15.tar.gz

Good:
* Upstream source tarball verified
* Package name conforms to the Fedora Naming Guidelines
* Group Tag is from the official list
* Buildroot has all required elements
* All paths begin with macros
* Builds fine in Mock (once missing BR (hal-devel) added to spec)
* Package installs and uninstalls cleanly on FC5.
* Make succeeds even when %{_smp_mflags} is defined

Bad:
* Missing Build Requirement for hal-devel
* Source URL isn't canonical. Should be
http://banshee-project.org/files/%{name}/%{name}-%{version}.tar.gz
* License is LGPL, not GPL.
* Add COPYING file to package.
* rpmlint gives the following errors:
  E: ipod-sharp no-binary
  E: ipod-sharp only-non-binary-in-usr-lib
  W: ipod-sharp no-documentation
  W: ipod-sharp devel-file-in-non-devel-package /usr/lib/pkgconfig/ipod-sharp.pc
  W: ipod-sharp devel-file-in-non-devel-package /usr/lib/pkgconfig/ipod-sharp-ui.pc

  Most of these errors can be ignored, or be easily corrected (like adding the
COPYING file).  You should probably add a -devel sub-package to handle the *.pc
files, though.

APPROVED, once these items are corrected.   

Comment 2 Brian Pepple 2006-03-10 19:54:05 UTC
This has been built hasn't it?  If so, this bug can be closed.

Comment 3 Christopher Aillon 2006-03-10 19:57:25 UTC
Yeah, I'm just lazy/forgetful.  ;-)


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