tuya-cloudcutter / bk7231tools

This is a collection of tools to interact with and analyze artifacts for BK7231 MCUs
MIT License
38 stars 6 forks source link

Missing PyCryptodome dependency #8

Closed narottamroyal closed 1 year ago

narottamroyal commented 1 year ago

Just noticed that storage.py uses the Cryptodome package, so I believe pyproject.toml should list pycryptodomex as a dependency.

Cossid commented 1 year ago

I think most of us testing this have been using a global pip installation. I have added it as suggested, can you please test/verify?

narottamroyal commented 1 year ago

I just tried to build/install in a fresh virtual environment, but poetry install failed because pycryptodomex is not specified in poetry.lock

Running the following command updated the lock file and allowed me to proceed with installation:

poetry lock --no-upgrade
Cossid commented 1 year ago

Ah, I didn't use the tool to add it to the dependency list. Did this time, and it looks like it was missing file metadata from a prior poetry bug. Should be fixed now.

narottamroyal commented 1 year ago

Yep, all sorted! 😊