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

mfg_tools requirements.txt is missing ecdsa module (CON-951) #788

Closed
gregbreen opened this issue Jan 3, 2024 · 4 comments
Closed

mfg_tools requirements.txt is missing ecdsa module (CON-951) #788

gregbreen opened this issue Jan 3, 2024 · 4 comments

Comments

@gregbreen
Copy link

Describe the bug
If you install the mfg_tool requirements.txt in a virtual environment, you will suffer the following error when you try to run the tool.

Traceback (most recent call last):
  File "/home/ubuntu/environment/esp-matter/tools/mfg_tool/./mfg_tool.py", line 49, in <module>
    from spake2p import generate_verifier
  File "/home/ubuntu/environment/esp-matter/connectedhomeip/connectedhomeip/scripts/tools/spake2p/spake2p.py", line 25, in <module>
    from ecdsa.curves import NIST256p
ModuleNotFoundError: No module named 'ecdsa'

Environment

  • ESP-Matter Commit Id: a7c1d96
  • ESP-IDF Commit Id: b6a66b7
@github-actions github-actions bot changed the title mfg_tools requirements.txt is missing ecdsa module mfg_tools requirements.txt is missing ecdsa module (CON-951) Jan 3, 2024
@shubhamdp
Copy link
Contributor

@gregbreen, thanks for raising the issue, initially, mfg-tool wasn't intended for standalone installation, so the ecdsa library is installed from this source.

We are currently in the process of releasing the mfg-tool to PyPi. Once completed, the mfg-tool can be installed using pip, and the ecdsa dependency will be handled seamlessly

@gregbreen
Copy link
Author

@shubhamdp thanks for the explanation. Please close this issue if it's not helpful.

@shubhamdp
Copy link
Contributor

@gregbreen We have pushed the mfg-tool here on pypi: https://pypi.org/project/esp-matter-mfg-tool/1.0.0 with all the resolved dependencies. Please give it a try.

@dhrishi dhrishi closed this as completed Feb 16, 2024
@gregbreen
Copy link
Author

Thank you!

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

3 participants