Skip to content

Update electrum-env #10018

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

Closed
wants to merge 1 commit into from
Closed

Update electrum-env #10018

wants to merge 1 commit into from

Conversation

PabloWong
Copy link

needed, or fail.

needed, or fail.
@SomberNight
Copy link
Member

This looks weird. Please provide more details what is failing for you.

Are you running Linux? Which distro and version?
What gets logged to stdout/stderr?
How is python installed? etc

@PabloWong
Copy link
Author

PabloWong commented Jul 11, 2025

The problem could have been solved by completely removing the export, but since it is there I chose not to touch it and added the local paths instead.

however:

os: debian 12
python installed with apt

./electrum-env 
Traceback (most recent call last):
  File "/home/pablowong/devel/electrum/./run_electrum", line 100, in <module>
    check_imports()
  File "/home/pablowong/devel/electrum/./run_electrum", line 89, in check_imports
    raise RuntimeError(f'aiorpcX version {aiorpcx._version} does not match required: 0.25.0<=ver<0.26')
RuntimeError: aiorpcX version (0, 22, 1) does not match required: 0.25.0<=ver<0.26
$ apt-cache show python3-aiorpcx
Package: python3-aiorpcx
Source: aiorpcx
Version: 0.22.1-2
Installed-Size: 140
Maintainer: Debian Python Team <[email protected]>
Architecture: all
Depends: python3:any
Suggests: python-aiorpcx-doc
Description-en: generic async RPC implementation, including JSON-RPC (Python 3)
 Transport, protocol and framing-independent async RPC client and server
 implementation.
 .
 This package installs the library for Python 3.
Description-md5: c1cbfb7d8b38834ce559133854b0af4e
Homepage: https://github.com/kyuupichan/aiorpcX
Section: python
Priority: optional
Filename: pool/main/a/aiorpcx/python3-aiorpcx_0.22.1-2_all.deb
Size: 31708
MD5sum: 41a15275d76958fa8de419a5b0c537f4
SHA256: 1f9cb57e219554b745616ded7be2c8360b6e56d2be774eac8aa9450347d0e4ab
$ pip show aiorpcx
Name: aiorpcX
Version: 0.25.0
Summary: Generic async RPC implementation, including JSON-RPC
Home-page:
Author:
Author-email: Neil Booth [[email protected]](mailto:[email protected])
License: Copyright (c) 2018 Neil Booth

All rights reserved.

The MIT License (MIT)

Permission is hereby granted, free of charge, to any person obtaining
a copy of this software and associated documentation files (the
"Software"), to deal in the Software without restriction, including
without limitation the rights to use, copy, modify, merge, publish,
distribute, sublicense, and/or sell copies of the Software, and to
permit persons to whom the Software is furnished to do so, subject to
the following conditions:

The above copyright notice and this permission notice shall be
included in all copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

Location: /home/pablowong/devel/electrum/env/lib/python3.11/site-packages
Requires:
Required-by: Electrum, electrum-aionostr

@@ -34,5 +34,6 @@ export PYTHONPATH="$PYTHONPATH:"\
"/usr/lib/python3/dist-packages:"\
"/usr/lib/python${PYTHON_VER}/site-packages:"

export PYTHONPATH="$PYHTONPATH:./env/lib/python${PYTHON_VER}/site-packages:./env/lib/python${PYTHON_VER}/dist-packages"
Copy link
Member

@SomberNight SomberNight Jul 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the only reason this "works" in your use case is because of the typo :P
PYHTONPATH (t<->h)

@SomberNight
Copy link
Member

The problem could have been solved by completely removing the export

Ok... I think modifying the PYTHONPATH is causing more troubles than it's worth.

We'll take a patch that removes all the PYTHONPATH-mangling, installs the gui and crypto extras by default, and modifies the comments in the rest of the file accordingly. (regarding "more extras" in line 11, the "hardware" extra should still be mentioned, but it should not be installed by default)

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

Successfully merging this pull request may close these issues.

2 participants