NordicSemiconductor / pynrfjprog

Python wrapper around the nrfjprog dynamic link library (DLL)
Other
74 stars 26 forks source link

python setup.py install error #8

Closed adison closed 6 years ago

adison commented 6 years ago

While on OSX, run setup.py with python 2.7.10 or python 3.6.2 failed with below message Does anything missed?

File "setup.py", line 19
    version = 0.0.1,
                  ^
SyntaxError: invalid syntax
Qbicz commented 6 years ago

Did you run python setup.py install on a current version? In the most recent version, version variable is a string: version = "9.6.0"

You can also install it from pip: pip install pynrfjprog

David-Garcia-Polo commented 6 years ago

Is the issue solved for you already?

adison commented 6 years ago

on OSX, pip2 install pynrfjprog works fine

adison commented 6 years ago

I just confirmed the issue is for nRF5-multi-prog. The version number of setup.py of nRF5-multi-prog should be wrapped by apostrophe. And the requirements of nRF5-multi-prog need a updated as well.

I have changed and tested on my repo, please take a look. https://github.com/adison/nRF5-multi-prog/commit/31615bdd143bad26a12fd5577c81f089f59d525c https://github.com/adison/nRF5-multi-prog/commit/88bed04ea4c5d86e2356db89c8c0b8ddfd2d0b4d

After there changes, pip install nRF5-multi-prog works fine.

My environment is OS X 10.12 with python 2.7.10