Comments

58 Comments

Has this been fixed since it was updated, @imaginaryusername ?

Yes, I noticed there is a PR already for qdarkstyle.

As for tox -- Why do you need to install all the pythons? I have tox installed on my system now (via pypi / aka pip) and I don't have all of the pythons. It seems to me distribution packages should be at least as good or better and producing lean and mean installs... no?

I don't know. I only noticed it when qdarkstyle started pulling in absolutely the entire 1.7GB Universe of Python. It made me realize the whole way that "recommends" get auto-installed is dangerously flawed.

Tox works with "whatever you go installed" already. There is no reason for it to recommend ALL OF THE PYTHONS. Nobody wants ALL OF THE PYTHONS.

Anyway you do what you like but please fix the qdarkstyle thing at least (in the other thread).

karma

why is this package pulling in ALL of the pythons as a "Recommends" dependency?! This ends up pulling in the WORLD when casually installing packages that pull this package in. Please fix this. This is a bug in the requirements for all variations of this package for FC 33, 34, 35, 36, etc.

Why does this project pull in tox as a dependency? tox is a testing suite for devs. It has no business in the final project. I only noticed because this pulls in like 1.4G of deps as a result (all of the pythons!!!).

Please fix. Tox is not a dependency of an installed qdarkstyle.

Runs great. No issues. Looks good too. Approved!

App tested on FC35 with all latest updates. Runs just fine for me, no issues.

Thanks Jonny

Thanks Jonny

Thanks Jonny

Installed, tested, works perfectly.

Can confirm, works as advertised (tested with Electron Cash).

Everything works great.

Works flawlessly. Even tor works. Nice job.

Ok, works.

Tested and works with Electron Cash 4.0.13-git.

All tests pass.

Tested and works with Electron Cash 4.0.13-git.

All tests pass.

Tested, works.

Tested, works.

Yes, everything works.