garecovery icon indicating copy to clipboard operation
garecovery copied to clipboard

Dependencies error

Open alex27riva opened this issue 1 year ago • 6 comments

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?

alex27riva avatar Dec 29 '23 21:12 alex27riva

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 avatar Dec 30 '23 01:12 jgriffiths

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.

jgriffiths avatar Dec 30 '23 01:12 jgriffiths

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.

alex27riva avatar Dec 30 '23 13:12 alex27riva

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

jgriffiths avatar Dec 30 '23 20:12 jgriffiths

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

jgriffiths avatar Dec 30 '23 20:12 jgriffiths

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.

alex27riva avatar Dec 30 '23 20:12 alex27riva