Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 602424 - net-p2p/deluge blocks net-proxy/mitmproxy because of dev-python/twisted
Summary: net-p2p/deluge blocks net-proxy/mitmproxy because of dev-python/twisted
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Paolo Pedroni
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-12-12 08:59 UTC by manwe
Modified: 2016-12-22 15:32 UTC (History)
2 users (show)

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


Attachments
0001-net-p2p-deluge-Fix-dependecy-on-dev-python-twisted-f.patch (0001-net-p2p-deluge-Fix-dependecy-on-dev-python-twisted-f.patch,3.13 KB, patch)
2016-12-15 10:42 UTC, Paolo Pedroni
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description manwe 2016-12-12 08:59:06 UTC
Net-p2p/deluge depends on dev-python/twisted-core and dev-python/twisted-web, while net-proxy/mitmproxy depends on dev-python/twisted.

You cannot have both dev-python/twisted and dev-python/twisted-{core,web}. 

Is there a way around it? 

 * Error: The above package list contains packages which cannot be
 * installed at the same time on the same system.

  (dev-python/twisted-core-15.2.1:0/0::gentoo, installed) pulled in by
    >=dev-python/twisted-core-13.0[python_targets_python2_7(-)?,-python_single_target_jython2_7(-),-python_single_target_pypy(-),-python_single_target_pypy3(-),-python_single_target_python3_4(-),-python_single_target_python3_5(-),python_single_target_python2_7(+)] (>=dev-python/twisted-core-13.0[python_targets_python2_7(-),-python_single_target_jython2_7(-),-python_single_target_pypy(-),-python_single_target_pypy3(-),-python_single_target_python3_4(-),-python_single_target_python3_5(-),python_single_target_python2_7(+)]) required by (net-p2p/deluge-1.3.13:0/0::gentoo, installed)
    =dev-python/twisted-core-15.2*[python_targets_python2_7(-)?,-python_single_target_python2_7(-)] (=dev-python/twisted-core-15.2*[python_targets_python2_7(-),-python_single_target_python2_7(-)]) required by (dev-python/twisted-web-15.2.1:0/0::gentoo, installed)

  (dev-python/twisted-16.4.1:0/0::gentoo, ebuild scheduled for merge) pulled in by
    >=dev-python/twisted-16.0.0[python_targets_python2_7(-)?,python_targets_python3_4(-)?,python_targets_python3_5(-)?,-python_single_target_python2_7(-),-python_single_target_python3_4(-),-python_single_target_python3_5(-)] (>=dev-python/twisted-16.0.0[python_targets_python2_7(-),python_targets_python3_4(-),-python_single_target_python2_7(-),-python_single_target_python3_4(-),-python_single_target_python3_5(-)]) required by (www-servers/tornado-4.4.2-r1:0/0::gentoo, ebuild scheduled for merge)

  (dev-python/twisted-web-15.2.1:0/0::gentoo, installed) pulled in by
    >=dev-python/twisted-web-13.0[python_targets_python2_7(-)?,-python_single_target_jython2_7(-),-python_single_target_pypy(-),-python_single_target_pypy3(-),-python_single_target_python3_4(-),-python_single_target_python3_5(-),python_single_target_python2_7(+)] (>=dev-python/twisted-web-13.0[python_targets_python2_7(-),-python_single_target_jython2_7(-),-python_single_target_pypy(-),-python_single_target_pypy3(-),-python_single_target_python3_4(-),-python_single_target_python3_5(-),python_single_target_python2_7(+)]) required by (net-p2p/deluge-1.3.13:0/0::gentoo, installed)
Comment 1 manwe 2016-12-12 09:08:17 UTC
I've removed dev-python/twisted-{core,web} and installed net-proxy/mitmproxy with dev-python/twisted. And deluge seems to be working fine with dev-python/twisted.
Comment 2 Paolo Pedroni 2016-12-14 15:48:36 UTC
I'll fix dependencies in deluge unstable ebuild to require:
|| ( >=dev-python/twisted-16.0.0[${PYTHON_USEDEP}] >=dev-python/twisted-web-13.0[${PYTHON_USEDEP}] )
as twisted-core is required by twisted-web anyway.

Stable deluge is going to stay as it is (I dare not touch it) as it will be superseded by 1.3.13 as soon as possible.

Kristian, is this fine for you? Does it look correct?
Comment 3 Kristian Fiskerstrand (RETIRED) gentoo-dev 2016-12-14 21:07:32 UTC
(In reply to Paolo Pedroni from comment #2)
> I'll fix dependencies in deluge unstable ebuild to require:
> || ( >=dev-python/twisted-16.0.0[${PYTHON_USEDEP}]
> >=dev-python/twisted-web-13.0[${PYTHON_USEDEP}] )
> as twisted-core is required by twisted-web anyway.
> 
> Stable deluge is going to stay as it is (I dare not touch it) as it will be
> superseded by 1.3.13 as soon as possible.
> 
> Kristian, is this fine for you? Does it look correct?

optional dep itself is correct, not sure how closely tied twisted-web and twisted-core is, it might still make sense to have a dep on both if there is uncertainty (just use an additional parenthesis) , but these are possibly so closely linked it is OK anyways.
Comment 4 Paolo Pedroni 2016-12-15 09:56:02 UTC
(In reply to Kristian Fiskerstrand from comment #3)
> optional dep itself is correct, not sure how closely tied twisted-web and
> twisted-core is, it might still make sense to have a dep on both if there is
> uncertainty (just use an additional parenthesis) , but these are possibly so
> closely linked it is OK anyways.

From twisted-web ebuild:
DEPEND="
        =dev-python/twisted-core-${TWISTED_RELEASE}*[${PYTHON_USEDEP}]

so it looks good to me.
Comment 5 Paolo Pedroni 2016-12-15 10:42:35 UTC
Created attachment 456288 [details, diff]
0001-net-p2p-deluge-Fix-dependecy-on-dev-python-twisted-f.patch

Patch to fix dependencies on twisted monolithic ebuilds.

k_f, please review and apply.
Comment 6 Paolo Pedroni 2016-12-22 11:14:35 UTC
Kristian, is everything all right? Is there some problem with the patch?
Comment 7 Kristian Fiskerstrand (RETIRED) gentoo-dev 2016-12-22 15:32:44 UTC
(In reply to Paolo Pedroni from comment #6)
> Kristian, is everything all right? Is there some problem with the patch?

I made a few changes, commited as:

commit c0c490b40ca22da7024d320bf6248b09ed72db92
Author: Kristian Fiskerstrand <k_f@gentoo.org>
Date:   Thu Dec 22 16:19:19 2016 +0100

    net-p2p/deluge: Clean up old testing versions
    
    Package-Manager: Portage-2.3.3, Repoman-2.3.1

commit 95fa09d1a3f307d7dda0bec6391153b7aa11eee4
Author: Kristian Fiskerstrand <k_f@gentoo.org>
Date:   Thu Dec 22 16:18:25 2016 +0100

    net-p2p/deluge: Optional dep on monolithic or modular twisted
    
    Gentoo-Bug: 602424
    
    Package-Manager: Portage-2.3.3, Repoman-2.3.1