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

unable to run master branch #6258

Closed
jlopp opened this issue Jun 23, 2020 · 6 comments
Closed

unable to run master branch #6258

jlopp opened this issue Jun 23, 2020 · 6 comments

Comments

@jlopp
Copy link

jlopp commented Jun 23, 2020

Running commit 040f66a

$ ./run_electrum 
Traceback (most recent call last):
  File "./run_electrum", line 84, in <module>
    from electrum.logging import get_logger, configure_logging
  File "/home/jameson/code/electrum/electrum/__init__.py", line 15, in <module>
    from .util import format_satoshis
  File "/home/jameson/code/electrum/electrum/util.py", line 49, in <module>
    from aiohttp_socks import ProxyConnector, ProxyType
ImportError: cannot import name 'ProxyConnector' from 'aiohttp_socks'

I tried upgrading pip, reinstalling dependencies, no luck.

$ python3 -m pip install --user -e .
Obtaining file:///home/jameson/code/electrum
Requirement already satisfied: aiohttp<4.0.0,>=3.3.0 in /usr/local/lib/python3.7/dist-packages (from Electrum==4.0.0b0) (3.5.1)
Requirement already satisfied: aiohttp_socks>=0.3 in /home/jameson/.local/lib/python3.7/site-packages (from Electrum==4.0.0b0) (0.3.9)
Requirement already satisfied: aiorpcx<0.19,>=0.18 in /home/jameson/.local/lib/python3.7/site-packages (from Electrum==4.0.0b0) (0.18.4)
Requirement already satisfied: attrs in /usr/local/lib/python3.7/dist-packages (from Electrum==4.0.0b0) (18.2.0)
Requirement already satisfied: bitstring in /home/jameson/.local/lib/python3.7/site-packages (from Electrum==4.0.0b0) (3.1.6)
Requirement already satisfied: certifi in /usr/lib/python3/dist-packages (from Electrum==4.0.0b0) (2018.8.24)
Requirement already satisfied: dnspython in /usr/local/lib/python3.7/dist-packages (from Electrum==4.0.0b0) (1.16.0)
Requirement already satisfied: ecdsa>=0.14 in /home/jameson/.local/lib/python3.7/site-packages (from Electrum==4.0.0b0) (0.15)
Requirement already satisfied: protobuf>=3.12 in /home/jameson/.local/lib/python3.7/site-packages (from Electrum==4.0.0b0) (3.12.2)
Requirement already satisfied: pyaes>=0.1a1 in /usr/local/lib/python3.7/dist-packages (from Electrum==4.0.0b0) (1.6.1)
Requirement already satisfied: qdarkstyle<2.9 in /usr/local/lib/python3.7/dist-packages (from Electrum==4.0.0b0) (2.5.4)
Requirement already satisfied: qrcode in /usr/local/lib/python3.7/dist-packages (from Electrum==4.0.0b0) (6.0)
Requirement already satisfied: async-timeout<4.0,>=3.0 in /usr/local/lib/python3.7/dist-packages (from aiohttp<4.0.0,>=3.3.0->Electrum==4.0.0b0) (3.0.1)
Requirement already satisfied: yarl<2.0,>=1.0 in /usr/local/lib/python3.7/dist-packages (from aiohttp<4.0.0,>=3.3.0->Electrum==4.0.0b0) (1.3.0)
Requirement already satisfied: multidict<5.0,>=4.0 in /usr/local/lib/python3.7/dist-packages (from aiohttp<4.0.0,>=3.3.0->Electrum==4.0.0b0) (4.5.2)
Requirement already satisfied: chardet<4.0,>=2.0 in /usr/lib/python3/dist-packages (from aiohttp<4.0.0,>=3.3.0->Electrum==4.0.0b0) (3.0.4)
Requirement already satisfied: six>=1.9.0 in /usr/lib/python3/dist-packages (from ecdsa>=0.14->Electrum==4.0.0b0) (1.12.0)
Requirement already satisfied: setuptools in /usr/lib/python3/dist-packages (from protobuf>=3.12->Electrum==4.0.0b0) (40.8.0)
Requirement already satisfied: idna>=2.0 in /usr/lib/python3/dist-packages (from yarl<2.0,>=1.0->aiohttp<4.0.0,>=3.3.0->Electrum==4.0.0b0) (2.6)
Installing collected packages: Electrum
  Attempting uninstall: Electrum
    Found existing installation: Electrum 4.0.0b0
    Uninstalling Electrum-4.0.0b0:
      Successfully uninstalled Electrum-4.0.0b0
  Running setup.py develop for Electrum
Successfully installed Electrum

This is on a PureOS machine (Debian 10)

@SomberNight
Copy link
Member

SomberNight commented Jun 23, 2020

Try deleting the packages directory in the project root. Do you have that?

That contains frozen dependencies, which have import priority. You would have this folder if you e.g. tried to build distributables previously.

@jlopp
Copy link
Author

jlopp commented Jun 23, 2020

Gotcha, that has changed my error:

$ ./run_electrum 
Traceback (most recent call last):
  File "./run_electrum", line 84, in <module>
    from electrum.logging import get_logger, configure_logging
  File "/home/jameson/code/electrum/electrum/__init__.py", line 16, in <module>
    from .wallet import Wallet
  File "/home/jameson/code/electrum/electrum/wallet.py", line 65, in <module>
    from .storage import StorageEncryptionVersion, WalletStorage
  File "/home/jameson/code/electrum/electrum/storage.py", line 36, in <module>
    from .wallet_db import WalletDB
  File "/home/jameson/code/electrum/electrum/wallet_db.py", line 36, in <module>
    from .invoices import PR_TYPE_ONCHAIN, Invoice
  File "/home/jameson/code/electrum/electrum/invoices.py", line 146, in <module>
    class LNInvoice(Invoice):
  File "/usr/local/lib/python3.7/dist-packages/attr/_make.py", line 904, in attrs
    return wrap(maybe_cls)
  File "/usr/local/lib/python3.7/dist-packages/attr/_make.py", line 855, in wrap
    cache_hash,
  File "/usr/local/lib/python3.7/dist-packages/attr/_make.py", line 471, in __init__
    cls, these, auto_attribs, kw_only
  File "/usr/local/lib/python3.7/dist-packages/attr/_make.py", line 416, in _transform_attrs
    a=a
ValueError: Non keyword-only attributes are not allowed after a keyword-only attribute.  Attribute in question: Attribute(name='invoice', default=NOTHING, validator=None, repr=True, cmp=True, hash=None, init=True, metadata=mappingproxy({}), type=<class 'str'>, converter=None, kw_only=False)

@SomberNight
Copy link
Member

Ok, so apparently we need python-attrs/attrs#559
so at least attrs 19.2
will update requirements

SomberNight added a commit that referenced this issue Jun 23, 2020
@SomberNight
Copy link
Member

try with a95738f
you can rerun the full pip install, or just manually update just this dependency

@jlopp
Copy link
Author

jlopp commented Jun 23, 2020

Resolved 👍

@jlopp jlopp closed this as completed Jun 23, 2020
@SomberNight
Copy link
Member

Thank you for testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants