[ Adding David to the CC list at his request and adding the python-libtorrent
dependency. ]

Compiles fine on both FC6 and Development however it doesn't actually work with
any torrents:

GTK Accessibility Module initialized
./deluge.py:62: GtkWarning: gtk_scrolled_window_add(): cannot add non scrollable
widget use gtk_scrolled_window_add_with_viewport() instead
  self.wTree = gtk.glade.XML(self.gladefile)
python-libtorrent, using libtorrent 0.11.0.0. Compiled with NDEBUG value: 1
Capping download to -1 bytes per second
Capping upload to 40960 bytes per second
Loading DHT state from /home/david/.config/deluge/dht.state
No DHT file to resume
opening torrent from URL
ok clicked: http://torrent.fedoraproject.org/torrents/Zod-dvd-x86_64.torrent
filename: /tmp/tmp9M8gRq.torrent
file is a torrent
Critical Error! No such uniqueID (46909632806912, 1)
terminate called after throwing an instance of 'std::out_of_range'
  what():  vector::_M_range_check
Afbrudt (SIGABRT)

definitely and sadly so, not Extras ready - I bugged upstream:
http://deluge-torrent.org/trac/ticket/95

deluge works great for me. AMD64 (x86_64)
It's fundamental to compile python-libtorrent with -DAMD64
In python-libtorrent.cpp are this lines
#ifdef AMD64
#define pythonLong int
#else
#define pythonLong long   <- see -DAMD64
#endif
I used this :
python-libtorrent-0.3.0-3.20061128svn.src.rpm
rb_libtorrent-0.11-2.src.rpm
deluge-0.4.1-1.src.rpm

the python-libtorrent.spec should changed for x86_64

Thanks for your help on getting this worked out, Gerd. I've posted an updated
python-libtorrent spec/SRPM on its review request (bug #221376), and I've bumped
the dependecy thereof in the spec/SRPM for Deluge as well. Update files are on
my webspace: 

Spec: http://thecodergeek.com/downloads/fedora/deluge.spec
SRPM: http://thecodergeek.com/downloads/fedora/deluge-0.4.1-2.src.rpm

Thanks.

i believe that you should avoid the 'find' command, just you should use
'install' command for change file access permissions in 'copy-time' with the
parameters '-p -m755' on all *.py in %{python_sitelib}/%{name}/

Haha. I completely forgot about the install command for a moment. T_T

I've updated the spec with your suggestion, Wilmer (which made it quite a bit
cleaner, actually ^_^).

SPEC: http://thecodergeek.com/downloads/fedora/deluge.spec
SRPM: http://thecodergeek.com/downloads/fedora/deluge-0.4.1-3.src.rpm

Thanks.

Umm..
on FC-devel, this program completely hangs up after printing
the following messages...

----------------------------------------------------------
[tasaka1@localhost i386]$ deluge
GTK Accessibility Module initialized
./deluge.py:62: GtkWarning: gtk_scrolled_window_add(): cannot add non scrollable
widget use gtk_scrolled_window_add_with_viewport() instead
  self.wTree = gtk.glade.XML(self.gladefile)
python-libtorrent, using libtorrent 0.11.0.0. Compiled with NDEBUG value: 1
Capping download to -1 bytes per second
Capping upload to -1 bytes per second
Loading DHT state from /home/tasaka1/.config/deluge/dht.state
No DHT file to resume
--------------------------------------------------------

On FC-5, it does not hang up. Would you know why?

( As python-libtorrent (bug 221376) seems to be accepted soon after
 rb_libtorrent (bug 221372) is completed, I assign this bug to
 myself. )

What do you mean by hanging up? Does it draw the window; but have it
unresponsive to input? Does it simply not function right? I'm using FC-6 (with a
lot of X-related stuff from Rawhide) and have seen no such problem. :(

Well, "hang up" means...

When I launch "deluge"
* the window is drawn, with characters on the window
* the message (in comment 7) is printed out
* but the window does not response to any mouse click anymore...

Same thing happens on Development (x86_64)

(By the way, I would appreciate if you would review my
 review request bug 219327)

Umm..

Actually this works well with FC-5 i386, however, still
hangs up on FC-devel i386...

INFORMATION:

When I remove at-spi and gail forcely, deluge works on FC-devel,
too.

Actually it seems that FC-devel gail are causing problems on
several packages. For me, two packages I maintain crash on FC-devel.

So I will restart reviewing on this package, assuming that
deluge freeze is not due to deluge itself. Please wait...

Ah.. X crashed when I was writing the review result...
Well,

* Desktop files
  - Icon does not appear on desktop menu.
------------------------------------------------
Icon=deluge-256.png
------------------------------------------------
    ... should be deluge.png

* Python related dependency
   - notify-python
------------------------------------------------
    33                  try:
    34                          import pynotify
    35                          self.pynotify = pynotify # We must save this,
because as a plugin, our globals will die
------------------------------------------------
     notify-python is available on FC6/devel (not on FC5)

I can confirm that if you disable a11y support on Development this works just
peachy except the fact that it really doesn't:

- clicking a torrent in Epiphany correctly offers to open with Deluge but the
torrent is infact not added to the deluge queue.
- It hangs quite frequently, after recovering from such a hang you have to chant
the magic verses and restart a few times or it will add one to the port you told
it to use and thus both spam libnotify messages and also run very slowly because
it doesn't pull over and ask for directions.
- It doesn't obey the GNOME default browser setting (or any platform I suspect),
when testing the port it starts up Firefox (the same is true for Translate this
application, which opens the unfree Launchpad service)
- Setting the bandwidth limitations is without notations (upload 40 what?), also
0 in those should be labelled unlimited.

I'll go bug upstream ever so much. However the frequent hanging makes me wonder
if this is ready to go in, you can hardly turn your back without it hanging and
it doesn't recover gracefully at all.

(In reply to comment #16)
> I can confirm that if you disable a11y support on Development this works just
> peachy except the fact that it really doesn't:
> 
Can you check if the phenomena you are seeing on FC-devel (perhaps
x86_64) also occur on FC-6/5?

I suspect that most of the issues you are seeing is due to
FC-devel gail or something.

Well, kazehakase upstream Ikezoe sent me a patch against
gail 1.10.1 and with the patch deluge doesn't seem to hang
up any longer, so I suspect the problem is due to gail,
not deluge more and more.

So Peter, would you fix the issue I commented on comment 15?

Hello; and sorry for not getting back to this bug earlier. I was swamped with
work and class stuff most of last week.  :(

I've updated the deluge spec and SRPM with your suggestions from comment 15; and
the files are on my webspace.

Spec: http://www.thecodergeek.com/downloads/fedora/SPECs/deluge.spec
SRPM: http://www.thecodergeek.com/downloads/fedora/SRPMs/deluge-0.4.1-4.src.rpm


David: I recently downloaded the entire FC6/x86_64 DVD image overnight via the
torrent using Deluge and received no such trouble. I suspect it is something in
Rawhide eating more babies. The other two issues are things that need to be
resolved upstream, and we'll just need to poke and prod them until they do. :)

Thanks.

Well I will want to check by tomorrow (in Japan: EST +14)...

(In reply to comment #19)
>> Spec: http://www.thecodergeek.com/downloads/fedora/SPECs/deluge.spec
> SRPM: http://www.thecodergeek.com/downloads/fedora/SRPMs/deluge-0.4.1-4.src.rpm

Well, 
* shebang
  - Well, lots of rpmlint complaints like:
----------------------------------------------------
E: deluge non-executable-script
/usr/lib/python2.5/site-packages/deluge/plugins/NetworkHealthMonitor.py 0644
----------------------------------------------------
    Please remove shebang unless the script should have
    executable permissions.

* Requires
  - well, "notify-python" is available on FC6, FC-devel, not on
    FC-5 (nor FE5)
  - And this works without notify-python with some limitation
    So
--------------------------------------------
%if 0%{?fedora} >= 6
Requires: notify-python
%endif
--------------------------------------------
    will be better.

Just a note:
On rawhide gail is updated to 1.17.0, however, still
the patch from Ikezoe is needed...

ping?

(In reply to comment #23)
> ping?

Pong. :]

I've posted updated files on my webspace which take care of the issues you mention.

* Wed Feb 21 2007 Peter Gordon <peter> - 0.4.1-5 
- Make notify-python dependency conditional (FC6+ only)
- Strip the unneeded shebang lines from the plugin scripts, since they are not
  meant to be directly executed.

Spec: http://thecodergeek.com/downloads/fedora/SPECs/deluge.spec
SRPM: http://thecodergeek.com/downloads/fedora/SRPMs/deluge-0.4.1-5.src.rpm

Thanks!

Okay.

One issue:
http://deluge.mynimalistic.net/downloads/deluge-0.4.1.tar.gz
does not work for me.
http://deluge-torrent.org/downloads/deluge-0.4.1.tar.gz works.

One comment:
It seems that 0.5 beta 1 is out. However it is beta version
and as this package can be used for FC6/5, in which branch
stable release should usually used, so I will accept 0.4.1
version.

----------------------------------------------------------
  This package (deluge) is APPROVED by me.
----------------------------------------------------------

Now the process of importing new packages changed, so please
recheck the process from step 8 of
http://fedoraproject.org/wiki/Extras/NewPackageProcess

I talked at lenght with the developer on the hang, he can't reproduce it on his
Ubuntu machine with the 5.0 codebase - so it seems that bug might go away once
5.0 is released and packaged.

(In reply to comment #26)
> I talked at lenght with the developer on the hang, he can't reproduce it on his
> Ubuntu machine with the 5.0 codebase - so it seems that bug might go away once
> 5.0 is released and packaged.

It is due to what version of gail the developer uses...

0.4.1 hangs up with gail-1.17.0, however with gail 1.17.0 with a patch
from Kazehakase developer applied deluge does not hang up.
I have not checked 5.0beta version yet, if I have some time I will
want to try it. Anyway this package is approved.

Thanks for your review and comments!

I'll fix the Source URL and import this when I get home from classes tonight.
Since I'm planning a Rawhide upgrade anyway, I'll play around with the 0.5 beta
over the coming weekend, as well.

New Package CVS Request
=======================
Package Name: deluge
Short Description: A Python BitTorrent client with support for UPnP and DHT
Owners: peter
Branches: FC-5, FC-6, and devel
InitialCC: None

branched

Thanks, Dennis and all!

Imported and built - so closing as NEXTRELEASE.

deluge-0.4.90.2-1.fc7 seems no problem agaist
fedora rawhide gail-1.17.0-1.fc7.

(In reply to comment #32)
> deluge-0.4.90.2-1.fc7 seems no problem agaist
> fedora rawhide gail-1.17.0-1.fc7.

Great news! Thanks for testing it.

I hate to say it but 0.4.90.2 fails for me, it starts up just fine but you can't
add torrents also the configuration from the older version isn't transitioned so
you could imagine this happening:

User opens Deluge 0.4.1
adds a bunch of stuff
user then upgrades to 0.5b2 (for whatever reason, say Peter pushes an update or
he upgrades to F7)
User restarts deluge and voila all his torrents are gone, his configuration is
screwed, he now hates Peter, Fedora and life.

We would probably rather avoid that.

Also when I built the rpm for 0.5b2 I got an x86_64 binary not a noarch like
with 0.4.1 - any reason for the change?

(In reply to comment #34)
> We would probably rather avoid that.

Unfortunately, it seems to function mostly normally, but not show the torrents
in the list. I'm trying to debug that now.


> Also when I built the rpm for 0.5b2 I got an x86_64 binary not a noarch like
> with 0.4.1 - any reason for the change?

Yes. With 0.4.1, the bindings to rb_libtorrent were in a separate package.
However, with the 0.5 betas, they are integrated into Deluge as a core python
module.

Please branch for F-8. Thanks. :)

cvs done.