search

YosysHQ / icestorm

Project IceStorm - Lattice iCE40 FPGAs Bitstream Documentation (Reverse Engineered)
ISC License
985 stars 224 forks source link

Makefile assumes python binary is `python3` #238

Closed xobs closed 3 years ago

xobs commented 4 years ago

The icebox Makefile calls python3, which is only the name of this program on Unix-like systems (according to PEP 394). The official executable name on other platforms, notably on Windows, is python, and so the build fails on those platforms.

xobs commented 4 years ago

It's easy enough to add a variable to config.mk that lets you specify what python is called, which can get icebox to compile. However, how much updating should we do?

Should icefuzz be updated to use $(PYTHON)? What about icetime? And what about various scripts such as mktest.py?

mmicko commented 3 years ago

Your fix have been merged, note change from PYTHON to PYTHON3