Makefile: run pyflakes as 'pyflakes', not python `which pyflakes`
authorBrian Warner <warner@lothar.com>
Fri, 5 Feb 2016 20:32:02 +0000 (12:32 -0800)
committerBrian Warner <warner@lothar.com>
Fri, 5 Feb 2016 20:32:02 +0000 (12:32 -0800)
This funky invocation syntax was introduced in 2007 (commit 56ad518),
with a comment of "make pyflakes run faster". I no longer have any idea
why that might have been the case. It's time to simplify this, because
some of our buildslaves have pipsi-installed "pyflakes" on their $PATH,
which use a "python" that's different than the one on $PATH.

Makefile

index 6df6bd6adf0b48358e7a45d35f4542462dbcf573..89e2b41b6911c905b0d701dfff86e90b0e56bbd9 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -5,6 +5,8 @@ default: build
 
 PYTHON=python
 export PYTHON
+PYFLAKES=pyflakes
+export PYFLAKES
 
 # setup.py will extend sys.path to include our support/lib/... directory
 # itself. It will also create it in the beginning of the 'develop' command.
@@ -132,7 +134,7 @@ check-miscaptures:
 
 .PHONY: pyflakes
 pyflakes:
-       @$(PYTHON) -OOu `which pyflakes` $(SOURCES) |sort |uniq
+       $(PYFLAKES) $(SOURCES) |sort |uniq
        @echo
 
 .PHONY: check-umids