Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Helpdev package became a mandatory dependency, make it optional #182

Closed
SomberNight opened this issue Jun 4, 2019 · 1 comment
Closed
Assignees
Milestone

Comments

@SomberNight
Copy link

The helpdev package was introduced as a mandatory dependency for 2.7, in #147
Along with helpdev, this pulls in importlib-metadata, psutil, and zipp as indirect dependencies.

It seems to me that helpdev is only used for diagnostics/bug reports.
Would it be possible to turn it into an optional dependency instead?

SomberNight added a commit to spesmilo/electrum that referenced this issue Jun 4, 2019
@dpizetta dpizetta added this to the 2.8 milestone Jun 4, 2019
@dpizetta
Copy link
Collaborator

dpizetta commented Jun 4, 2019

We can study this in the v2.8, as we will introduce qtpy as mandatory to simplify many things in the code (as seen in the develop branch). As we won't use directly the bindings, the other functions of helpdev won't be needed except for reports.

@dpizetta dpizetta self-assigned this Jun 4, 2019
cculianu added a commit to Electron-Cash/Electron-Cash that referenced this issue Jun 4, 2019
Taken from spesmilo#046518d7f73fdee30fb974b283780fc4691a614e

2.6.8 is well tested and looks good. Other earlier versions may have
bugs.

QDarkStyle 2.7 pulls in some dependencies, see
ColinDuquesnoy/QDarkStyleSheet#182

Ergo, we restrict to exactly 2.6.8
cculianu added a commit to simpleledger/Electron-Cash-SLP that referenced this issue Jun 4, 2019
Taken from spesmilo#046518d7f73fdee30fb974b283780fc4691a614e

2.6.8 is well tested and looks good. Other earlier versions may have
bugs.

QDarkStyle 2.7 pulls in some dependencies, see
ColinDuquesnoy/QDarkStyleSheet#182

Ergo, we restrict to exactly 2.6.8
@dpizetta dpizetta changed the title helpdev package became a mandatory dependency. asking to make it optional Helpdev package became a mandatory dependency, make it optional Jun 5, 2019
@dpizetta dpizetta modified the milestones: 2.8, 2.9 Jun 21, 2019
SomberNight added a commit to SomberNight/electrum that referenced this issue Jun 3, 2020
Previously we stuck with version 2.6.8 as that had no deps but later
versions introduced several deps. However, now latest version only
has two dependencies (one of which has the same maintainer).
Futher, there are some bugs with 2.6.8 when used with new Qt,
e.g. with dropdowns that I want fixed (which it is in the newer ones).

related ColinDuquesnoy/QDarkStyleSheet#182
SomberNight added a commit to spesmilo/electrum that referenced this issue Jun 3, 2020
Previously we stuck with version 2.6.8 as that had no deps but later
versions introduced several deps. However, now latest version only
has two dependencies (one of which has the same maintainer).
Futher, there are some bugs with 2.6.8 when used with new Qt,
e.g. with dropdowns that I want fixed (which it is in the newer ones).

related ColinDuquesnoy/QDarkStyleSheet#182
dpizetta added a commit that referenced this issue Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants