meejah / txtorcon

Twisted-based asynchronous Tor control protocol implementation. Includes unit-tests, examples, state-tracking code and configuration abstraction.
http://fjblvrw2jrxnhtg67qpbzi45r7ofojaoo3orzykesly2j3c2m3htapid.onion/
MIT License
249 stars 72 forks source link

Drop or declare dependency on six #395

Closed dvzrv closed 5 months ago

dvzrv commented 6 months ago

Hi! :wave:

I am currently looking into the package on Arch Linux for this project.

I noticed that a few corners in the code exists, where six is being used. As this project only targets Python > 3 it would probably be great to remove this (implicit dependency).

In the meantime it might be good to specify six as a dependency, as it is only pulled in implicitly via automat (and if that ever changes, downstreams will be broken).

dvzrv commented 6 months ago

Looking at the automat upstream, they do not in fact rely on six anymore and our packaging just does not yet reflect that.

meejah commented 6 months ago

Yes, and six usage can / should be dropped in txtorcon ... would accept a PR (or can hopefully do that myself later this week)

a-detiste commented 5 months ago

done