Arch Linux

Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines

Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.

REPEAT: Do NOT report bugs for outdated packages!
Tasklist

FS#31433 - [deluge] move storage does not work

Attached to Project: Arch Linux
Opened by Byron Doyle (greensquares) - Thursday, 06 September 2012, 03:14 GMT
Last edited by Gerardo Exequiel Pozzi (djgera) - Saturday, 17 November 2012, 04:29 GMT
Task Type Bug Report
Category Upstream Bugs
Status Closed
Assigned To Jan Alexander Steffens (heftig)
Architecture x86_64
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

Description:

Moving storage does not work as described in this ticket for the Ubuntu package:

https://bugs.launchpad.net/ubuntu/+source/deluge/+bug/1041415

The fix was described as

"Add patch to fix function parameter on call to libtorrent to move storage.
It doesn't want Unicode (LP: #1041415)."

Additional info:

Package version 1.3.5-1


Steps to reproduce:

Attempt to move storage on a downloaded torrent OR set deluged to automatically move storage on completion of download, wait for download to finish.
This task depends upon

Closed by  Gerardo Exequiel Pozzi (djgera)
Saturday, 17 November 2012, 04:29 GMT
Reason for closing:  Upstream
Comment by Alexander F. Rødseth (xyproto) - Saturday, 08 September 2012, 23:10 GMT
Do you happen to know if this has been reported upstream?
Comment by Jan Alexander Steffens (heftig) - Saturday, 08 September 2012, 23:23 GMT
It has. http://dev.deluge-torrent.org/ticket/2140
Comment by Alexander F. Rødseth (xyproto) - Saturday, 08 September 2012, 23:29 GMT
Waiting for a new release, then. Thx.
Comment by Fredrik (vEX) - Tuesday, 23 October 2012, 17:27 GMT
The ticket has been closed for ~4 weeks now. Looking at their trac it seems that the next release isn't ready for some time yet (20 fixed tickets and 39 pending ones). Perhaps this could be patched in the Arch repo?
Comment by Mikael Ahlnider (emaz) - Saturday, 03 November 2012, 09:24 GMT
  • Field changed: Percent Complete (100% → 0%)
Still does not work for auto move with labels when torrent is complete.

That feature sends the dest parameter with unicode encoding.
The call to the libtorrent move_storage expects acsii encoded string.
Comment by Alexander F. Rødseth (xyproto) - Sunday, 04 November 2012, 15:22 GMT
@Fredrik If it takes too long for a new release that fixes issues, it's up to the maintainer to apply patches temporarily (not really "the Arch Way"), pester upstream until they make a new release (the "proper Arch Way" ;) ) or move the package to unsupported. If there are problems that aren't fixed, it is, in effect, unmaintained, and it's just a question of how long to wait.
Comment by Mikael Ahlnider (emaz) - Friday, 09 November 2012, 11:02 GMT
If you are interested, this is my change in torrent.py (function move_storage) that works for me:
(last row is the fix).

if deluge.common.windows_check():
# Attempt to convert utf8 path to unicode
# Note: Inconsistent encoding for 'dest', needs future investigation
try:
dest_u = unicode(dest, "utf-8")
except TypeError:
# String is already unicode
dest_u = dest
else:
dest_u = dest.encode('ascii','ignore')
Comment by Jan Alexander Steffens (heftig) - Friday, 09 November 2012, 19:53 GMT
If the patches I pulled from upstream do not work, please get upstream to commit a fix. I'm not going to add patches that upstream hasn't accepted. That way it won't get fixed properly.

Loading...