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 452842 - Review Request: monotorrent - a bit torrent client for mono
Summary: Review Request: monotorrent - a bit torrent client for mono
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: David Nielsen
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2008-06-25 12:43 UTC by Paul F. Johnson
Modified: 2008-08-28 08:54 UTC (History)
4 users (show)

Fixed In Version: monotorrent-0.4-2.fc10
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2008-08-28 08:54:04 UTC
gnomeuser: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Paul F. Johnson 2008-06-25 12:43:00 UTC
Spec URL:
monotorrent is a bittorrent client for mono.

Comment 1 Michal Nowak 2008-06-25 20:20:10 UTC
Informal package review:

-Group:          Development/Languages
-License:        X11/MIT
+Group:          Applications/Internet
+License:        MIT

* Found group which fits better.
* claims X11 to be MIT
* NOTE: In sources is MIT but in COPYRIGHT file is GNU GPLv2. From my POV: It'd
be good to contact author and ask him to be clear on this.
 # Bug 241850 - no Mono on pp64
-ExclusiveArch: %ix86 x86_64 ppc ia64 armv4l sparc alpha
+ExcludeArch:    ppc64

* I guess this is saner.
-Monotorrent is a bittorrent for mono
+Monotorrent is an open source bittorrent library. The main aim of this 
+library is to provide a rich API for developers to connect to so they 
+can harness BitTorrent for all downloading needs. 

* Some more descriptions.

 %package devel
-Summary: .pc file for monotorrent
+Summary: Development files for for monotorrent
 Group: Documentation
 Requires: %{name} = %{version}-%{release} pkgconfig

 %description devel
-Package config file for monotorrent
+Development files for for monotorrent

* Just a strings.
 %setup -q
@@ -47,6 +49,7 @@ make DESTDIR=%{buildroot} install
 %files devel

* This is necessary to have rpmlint silent.

[root@dhcp-lab-192 SPECS]# rpmlint monotorrent.spec
monotorrent.i386: E: no-binary
monotorrent.i386: E: only-non-binary-in-usr-lib
monotorrent-devel.i386: W: no-documentation
3 packages and 1 specfiles checked; 2 errors, 1 warnings.

* Take a look at this. Complaint of the /usr/bin/... stuff not to be binaries
(they are shell scripts).
* Dunno about the /usr/lib/ stuff. Hope someone experienced will see this BZ.

Please incorporate changes. Feel free to ask.

Comment 2 David Nielsen 2008-06-28 12:11:40 UTC
I'm back from vacation in Brazil now, so I will take this one. Review coming
shortly, I promise.

Comment 3 David Nielsen 2008-06-28 12:41:18 UTC
Thank you Paul for packaging up this, any hope for Monsoon in Fedora?

Since Michal already did fine work:

%defattr(-,root,root,-) needed for the devel package and the descriptions and
summaries need to be expanded a bit for clarity aside that this builds in mock
(rawhide-x86_64) and otherwise stands up to the requirements.

Clear up the license conflict as pointed out by Mr. Nowak.

The exclusive line is a standard mono thing, we only have mono on certain
platforms and currently ppc64 is unsupported so we note it by removing it from
that exclusivearch line. It's not the clearest solution we know but it's
intended this way. The rpmlint stuff is standard, rpmlint doesn't know all it
needs to do to ignore this warning for mono packages - we tend to just ignore it

I'll approve this, pending the license being worked out, Fedora makes no
guanteees as to the license tag so we can ship with mistakes and the rest of the
problems can be fixed up at import time.


Comment 4 Kevin Fenzi 2008-06-29 04:25:09 UTC
Paul: Can you add a cvs template here with what you want in your cvs request?


Comment 5 Paul F. Johnson 2008-07-08 18:02:49 UTC
New Package CVS Request
Package Name: monotorrent
Short Description: Bittorrent client for .NET
Owners: pfj
Branches: f8, f9, rawhide
Cvsextras Commits: yes

Comment 6 Kevin Fenzi 2008-07-09 16:53:30 UTC
cvs done.

Comment 7 Michal Nowak 2008-07-22 12:40:10 UTC
You might want to close this BZ once is monotorrent live.

Comment 8 David Nielsen 2008-08-03 12:39:29 UTC
Paul do you plan to ever build this?

Comment 9 David Nielsen 2008-08-28 08:54:04 UTC
Okay spot built it on one of his license tag sprees

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