build: don't auto-destroy existing configuration
authorBen Noordhuis <info@bnoordhuis.nl>
Sat, 10 Aug 2013 11:00:51 +0000 (13:00 +0200)
committerBen Noordhuis <info@bnoordhuis.nl>
Sat, 10 Aug 2013 11:13:46 +0000 (13:13 +0200)
Don't run configure when the configure script has been touched. Doing so
would be okay if the Makefile passed the original arguments to configure
but it doesn't - it runs configure without any arguments, effectively
destroying the current configuration.

Remove this misfeature and instead print an error message telling the
user to (re-)run configure.

Makefile

index 32ebc78..d3d302b 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -53,7 +53,11 @@ else
 endif
 
 config.gypi: configure
-       $(PYTHON) ./configure
+       if [ -f $@ ]; then
+               $(error Stale $@, please re-run ./configure)
+       else
+               $(error No $@, please run ./configure first)
+       fi
 
 install: all
        $(PYTHON) tools/install.py $@ '$(DESTDIR)' '$(PREFIX)'