greenaddress / garecovery

GreenAddress recovery tool for 2of2 and 2of3 wallets
MIT License
107 stars 53 forks source link

Dependencies error #57

Open alex27riva opened 8 months ago

alex27riva commented 8 months ago

Hi everyone, I'm testing garecovery script on Fedora 39, Python 3.12.1 but I have the following error with dependencies:

➜  garecovery git:(master) source venv/bin/activate
(venv) ➜  garecovery git:(master) pip install --require-hashes -r tools/requirements.txt
Collecting argcomplete==1.8.2 (from -r tools/requirements.txt (line 1))
  Downloading argcomplete-1.8.2-py2.py3-none-any.whl (34 kB)
Collecting python-bitcoinrpc==1.0 (from -r tools/requirements.txt (line 2))
  Downloading python-bitcoinrpc-1.0.tar.gz (13 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
ERROR: Could not find a version that satisfies the requirement wallycore==0.8.4 (from versions: 0.8.1, 0.8.2, 1.0.0)
ERROR: No matching distribution found for wallycore==0.8.4

Is this script still maintained?

jgriffiths commented 8 months ago

Hi @alex27riva, thanks for reporting this issue. It appears PyPI has removed the (admittedly ancient) wally version we used. Could you try with the following PR? https://github.com/greenaddress/garecovery/pull/58

jgriffiths commented 8 months ago

FYI the latest Python we currently test with is 3.10, if you have any issue with the upgraded wally PR above please LMK and I'll attempt to setup a 3.12 env to test with.

alex27riva commented 8 months ago

I successfully built garecovery on a Debian 12 container (Python 3.11.2) using Distrobox. On Fedora I still get an error, but I think it is not supported since the instructions in the readme are only for Debian / Ubuntu.

jgriffiths commented 8 months ago

OK, thanks for the update. I'll keep this issue open and see if Fedora/Python12 support is possible in the new year.

jgriffiths commented 8 months ago

Oh @alex27riva did you use the above PR? I'd like to merge it if it works for you.

alex27riva commented 8 months ago

Oh @alex27riva did you use the above PR? I'd like to merge it if it works for you.

Yes, I think you can merge.