Make torrequest (and other deps) optional about sherlock

[ppfeister]

torrequest wasn't available on Fedora / EPEL either. Seems like a dead project and I don't want to package something 6 years out of date. Here's my patch file that I'm using to remove it from my own build --

https://github.com/ppfeister/pkg/blob/master/sherlock/0001-Remove-tor.patch

Note that my patch is against the new package, #2111, which is pending code review

Removing features this way isn't ideal, but it's a fine short term fix in my opinion. I would like to figure something out for torrequest soon but that's my own fix for packaging in the meantime

I remember looking into removing stem but I don't remember why I didn't....... will have to investigate now. It may be a depend of another depend or something...

exrex has been removed from the dependency list in #2127, although it's very likely that I'll be adding it back for other tests. If I do, it'll be a dev depend and not in the default group. Should be gtg there either way.

[mjsir911]

Removing features this way isn't ideal

Yeah, I'm interested in something much less destructive. Literally just moving the import would make things function until --tor is passed. Which is an acceptable tradeoff considering the app would fail anyways if tor isn't installed.