Ticket #3696 enhancement new

Opened 4 years ago

Last modified 3 years ago

Add extras_require to top-level setup.py to list Twisted optional dependancies

Reported by: thijs Owned by:
Priority: normal Milestone:
Component: core Keywords:
Cc: zooko Branch:
Author: Launchpad Bug:

Description

Setuptools provides a feature for listing optional dependancies:  extras_require. It seems like this functionality should be in trunk/twisted/python/dist.py or trunk/twisted/python/release.py or something, especially given that trunk/twisted/conch/topfiles/setup.py needs to know about the pycrypto dependency somehow too.

Change History

1

Changed 4 years ago by glyph

  • owner changed from glyph to radix

For previous discussion, see #976, which spawned this ticket.

2

Changed 4 years ago by glyph

Oops I meant #967.

3

Changed 3 years ago by zooko

See also #3238 (patch: declare that twisted requires pywin32 if it is to offer process management or iocp reactor on Windows).

4

Changed 3 years ago by zooko

See also  http://allmydata.org/trac/tahoe/ticket/774 (pycrypto package is required for manhole). We're going to declare that Tahoe-LAFS depends on pycrypto. But that isn't really right. Really it is that Tahoe-LAFS depends on Twisted's "manhole" feature, which depends on pycrypto. So for example, if a future release of Twisted implemented manhole without using pycrypto, then Tahoe-LAFS would not require pycrypto anymore. So the "best" way to solve this is for Tahoe-LAFS to specify that it depends on Twisted plus the "extra" of manhole, and for Twisted to declare that if you want the "extra" of manhole then you need pycrypto.

5

Changed 3 years ago by zooko

  • cc zooko added

6

Changed 3 years ago by davidsarah

Also, twisted.conch depends on pycrypto and pyasn1.

7

Changed 3 years ago by zooko

Opened #4438 which emphasizes that if Twisted fails to declare an existing dependency then it is a bug, but if Twisted fails to declare a new dependency then it is a regression.

8

Changed 2 years ago by <automation>

  • owner radix deleted
Note: See TracTickets for help on using tickets.