Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 636860 - net-p2p/deluge-1.3.15-r2 - relax libtorrent-rasterbar version dep
Summary: net-p2p/deluge-1.3.15-r2 - relax libtorrent-rasterbar version dep
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Paolo Pedroni
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-11-08 11:41 UTC by Paolo Pedroni
Modified: 2018-02-04 17:41 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
0001-net-p2p-deluge-fix-libtorrent-rasterbar-dependency-a.patch (0001-net-p2p-deluge-fix-libtorrent-rasterbar-dependency-a.patch,5.64 KB, patch)
2017-11-08 11:49 UTC, Paolo Pedroni
Details | Diff
0002-net-p2p-deluge-fix-libtorrent-rasterbar-dependency-a.patch (0002-net-p2p-deluge-fix-libtorrent-rasterbar-dependency-a.patch,1.57 KB, patch)
2017-11-08 11:50 UTC, Paolo Pedroni
Details | Diff
0003-net-p2p-deluge-files-folder-cleanup.patch (0003-net-p2p-deluge-files-folder-cleanup.patch,4.25 KB, patch)
2017-11-08 11:50 UTC, Paolo Pedroni
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Paolo Pedroni 2017-11-08 11:41:45 UTC
Contrary to what was claimed in bug #619812 deluge works fine with libtorrent-rasterbar-1.1.x. The patches I will attach thus allow net-p2p/deluge to build with any version of libtorrent-rasterbar.

On the other hand libtorrent-rasterbar-1.1 removed geoip support, but deluge added its own, based on dev-python/geoip-python. These patches change the USE=geoip dependency accordingly.

Sadly geoip-python is not supported on arm, so I need to remove ~arm keyword for deluge from 1.3.15-r2 onwards.

I applied the same changes (and some more tweaks) to the stable live ebuild.

I also add a patch cleaning up the 'files' folder.

Please, Kristian (or anyone in proxy-mantainer), review and apply. Thanks.
Comment 1 Paolo Pedroni 2017-11-08 11:49:40 UTC
Created attachment 503090 [details, diff]
0001-net-p2p-deluge-fix-libtorrent-rasterbar-dependency-a.patch
Comment 2 Paolo Pedroni 2017-11-08 11:50:07 UTC
Created attachment 503092 [details, diff]
0002-net-p2p-deluge-fix-libtorrent-rasterbar-dependency-a.patch
Comment 3 Paolo Pedroni 2017-11-08 11:50:38 UTC
Created attachment 503094 [details, diff]
0003-net-p2p-deluge-files-folder-cleanup.patch
Comment 4 Paolo Pedroni 2017-12-19 10:22:51 UTC
Please, someone in proxy-maint@g.o, review and apply. It's been more than one month...
Comment 5 Larry the Git Cow gentoo-dev 2017-12-20 22:13:55 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=fc9581d755e96bfd4eef2eb91669559f6d87ea0f

commit fc9581d755e96bfd4eef2eb91669559f6d87ea0f
Author:     PPed72 <paolo.pedroni@iol.it>
AuthorDate: 2017-11-08 11:45:35 +0000
Commit:     Kristian Fiskerstrand <k_f@gentoo.org>
CommitDate: 2017-12-20 22:13:47 +0000

    net-p2p/deluge: 'files' folder cleanup
    
    Bug: https://bugs.gentoo.org/636860

 net-p2p/deluge/files/deluge-web.service | 11 -----
 net-p2p/deluge/files/deluged.conf       |  9 -----
 net-p2p/deluge/files/deluged.init       | 71 ---------------------------------
 net-p2p/deluge/files/deluged.service    | 13 ------
 4 files changed, 104 deletions(-)

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=4f871b6795b07cf61dba87992172daacc46fe32a

commit 4f871b6795b07cf61dba87992172daacc46fe32a
Author:     PPed72 <paolo.pedroni@iol.it>
AuthorDate: 2017-11-08 11:43:46 +0000
Commit:     Kristian Fiskerstrand <k_f@gentoo.org>
CommitDate: 2017-12-20 22:13:45 +0000

    net-p2p/deluge: fix libtorrent-rasterbar dependency and geoip USE-dep
    
    Some more ebuild cleanup
    Bug: https://bugs.gentoo.org/636860

 net-p2p/deluge/deluge-1.3.9999.ebuild | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ce1b6a743f2153ee351873382a4ddf4df79c0dac

commit ce1b6a743f2153ee351873382a4ddf4df79c0dac
Author:     PPed72 <paolo.pedroni@iol.it>
AuthorDate: 2017-11-08 11:41:53 +0000
Commit:     Kristian Fiskerstrand <k_f@gentoo.org>
CommitDate: 2017-12-20 22:13:43 +0000

    net-p2p/deluge: fix libtorrent-rasterbar dependency and geoip USE-dep
    
    Bug: https://bugs.gentoo.org/636860

 net-p2p/deluge/deluge-1.3.15-r2.ebuild | 144 +++++++++++++++++++++++++++++++++
 1 file changed, 144 insertions(+)}
Comment 6 Paolo Pedroni 2017-12-21 15:20:38 UTC
Thanks, Kristian.
Comment 7 Larry the Git Cow gentoo-dev 2018-02-04 17:41:27 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2895e463bbc04c17e05806075abffcd47046605e

commit 2895e463bbc04c17e05806075abffcd47046605e
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2018-02-04 17:07:18 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-02-04 17:38:54 +0000

    net-p2p/deluge: Drop ~arm <1.3.15-r2, restore ~arm >=1.3.15-r2
    
    This was holding up net-libs/libtorrent-rasterbar cleanup.
    Cleanup duplicate KEYWORDS line in 9999 conditional block.
    
    Bug: https://bugs.gentoo.org/636860
    Package-Manager: Portage-2.3.24, Repoman-2.3.6

 net-p2p/deluge/deluge-1.3.15-r1.ebuild | 5 ++---
 net-p2p/deluge/deluge-1.3.15-r2.ebuild | 5 ++---
 net-p2p/deluge/deluge-1.3.15.ebuild    | 5 ++---
 net-p2p/deluge/deluge-1.3.9999.ebuild  | 3 +--
 net-p2p/deluge/deluge-9999.ebuild      | 3 +--
 5 files changed, 8 insertions(+), 13 deletions(-)

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ffedae04bc76c7304854ee20e667bc121ff60bc3

commit ffedae04bc76c7304854ee20e667bc121ff60bc3
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2018-02-04 16:56:36 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-02-04 17:38:54 +0000

    profiles: arm: Mask net-p2p/deluge[geoip]
    
    This enables us to restore ~arm keyword.
    
    Bug: https://bugs.gentoo.org/636860

 profiles/arch/arm/package.use.mask | 4 ++++
 1 file changed, 4 insertions(+)}