Skip to content

Update requirements.txt to allow installation on Windows - #7

Open
maxgerhardt wants to merge 2 commits into
stemnic:masterfrom
maxgerhardt:patch-1
Open

Update requirements.txt to allow installation on Windows#7
maxgerhardt wants to merge 2 commits into
stemnic:masterfrom
maxgerhardt:patch-1

Conversation

@maxgerhardt

Copy link
Copy Markdown

Cloning this repo on Windows and trying to pip install -r requirements.txt the file leads to a build error with an outdated Cython version. Upgrading Cython to the latest version was however not allowed because of the used pyedbglib having a < .. version constraint, so its version was updated to. Thus this tool then installs sucessfully.

Cloning this repo on Windows and trying to `pip install -r requirements.txt` the file leads to a build error with an outdated Cython version. Upgrading Cython to the latest version was however not allowed because of the used pyedbglib having a `< ..` version constraint, so its version was updated to. Thus this tool then installs sucessfully.
@maxgerhardt

Copy link
Copy Markdown
Author

The diff looks weird because Github has transcoded from UTF16-LE to UTF8 -- see normal file at

https://github.com/maxgerhardt/pyAVRdbg/blob/patch-1/requirements.txt

@mraardvark

Copy link
Copy Markdown

Should cython and hidapi not be removed entirely from requirements since they will come along with pyedbglib anyway?

@maxgerhardt

maxgerhardt commented Feb 12, 2021

Copy link
Copy Markdown
Author

You are right, with the requirements.txt

astroid==2.4.2
colorama==0.4.3
intelhex==2.2.1
isort==4.3.21
lazy-object-proxy==1.4.3
mccabe==0.6.1
pyedbglib==2.17.7.70 
pylint==2.5.3
pymcuprog==3.1.3.60
pyserial==3.4
six==1.15.0
toml==0.10.1
wrapt==1.12.1

(and a previous pip uninstall -r requirements.txt) it can pip install -r requirements.txt` without errors.


C:\Users\Max\pyAVRdbg>pip install -r requirements.txt
Collecting astroid==2.4.2
  Using cached astroid-2.4.2-py3-none-any.whl (213 kB)
Collecting colorama==0.4.3
  Using cached colorama-0.4.3-py2.py3-none-any.whl (15 kB)
Collecting intelhex==2.2.1
  Using cached intelhex-2.2.1-py2.py3-none-any.whl (50 kB)
Collecting isort==4.3.21
  Using cached isort-4.3.21-py2.py3-none-any.whl (42 kB)
Collecting lazy-object-proxy==1.4.3
  Using cached lazy_object_proxy-1.4.3-cp38-cp38-win_amd64.whl (21 kB)
Collecting mccabe==0.6.1
  Using cached mccabe-0.6.1-py2.py3-none-any.whl (8.6 kB)
Collecting pyedbglib==2.17.7.70
  Using cached pyedbglib-2.17.7.70-py3-none-any.whl (60 kB)
Collecting pylint==2.5.3
  Using cached pylint-2.5.3-py3-none-any.whl (324 kB)
Collecting pymcuprog==3.1.3.60
  Using cached pymcuprog-3.1.3.60-py3-none-any.whl (130 kB)
Collecting pyserial==3.4
  Using cached pyserial-3.4-py2.py3-none-any.whl (193 kB)
Collecting six==1.15.0
  Using cached six-1.15.0-py2.py3-none-any.whl (10 kB)
Collecting toml==0.10.1
  Using cached toml-0.10.1-py2.py3-none-any.whl (19 kB)
Collecting wrapt==1.12.1
  Using cached wrapt-1.12.1.tar.gz (27 kB)
Collecting cython
  Using cached Cython-0.29.21-cp38-cp38-win_amd64.whl (1.7 MB)
Collecting hidapi
  Downloading hidapi-0.10.1-cp38-cp38-win_amd64.whl (43 kB)
     |████████████████████████████████| 43 kB 3.4 MB/s
Requirement already satisfied: setuptools>=19.0 in c:\users\max\appdata\local\programs\python\python38\lib\site-packages (from hidapi->pyedbglib==2.17.7.70->-r requirements.txt (line 7)) (49.2.1)
Building wheels for collected packages: wrapt
  Building wheel for wrapt (setup.py) ... done
  Created wheel for wrapt: filename=wrapt-1.12.1-cp38-cp38-win_amd64.whl size=33695 sha256=7b6d92c50e029865d5be51c0ccd2c2caf7d6fd4c727c8ac6a6eb2ed2ab3655bb
  Stored in directory: c:\users\max\appdata\local\pip\cache\wheels\5f\fd\9e\b6cf5890494cb8ef0b5eaff72e5d55a70fb56316007d6dfe73
Successfully built wrapt
Installing collected packages: wrapt, six, pyserial, lazy-object-proxy, hidapi, cython, toml, pyedbglib, mccabe, isort, intelhex, colorama, astroid, pymcuprog, pylint
Successfully installed astroid-2.4.2 colorama-0.4.3 cython-0.29.21 hidapi-0.10.1 intelhex-2.2.1 isort-4.3.21 lazy-object-proxy-1.4.3 mccabe-0.6.1 pyedbglib-2.17.7.70 pylint-2.5.3 pymcuprog-3.1.3.60 pyserial-3.4 six-1.15.0 toml-0.10.1 wrapt-1.12.1

I guess if you do a test on linux it's OK. I updated the PR.

Rmove Cython and hidapi
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