correcting the makefile

[youtube-dl] / Makefile

diff --git a/Makefile b/Makefile
index 21b34690523cd13e0e5eac6192135659faf92008..9f8dc6d938405ccd228b77074130219cb2b32347 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -1,4 +1,8 @@
-all: youtube-dl youtube-dl.exe README.md youtube-dl.1 youtube-dl.bash-completion LATEST_VERSION
+all: youtube-dl README.md youtube-dl.1 youtube-dl.bash-completion LATEST_VERSION
+# TODO: re-add youtube-dl.exe, and make sure it's 1. safe and 2. doesn't need sudo
+
+clean:
+       rm -f youtube-dl youtube-dl.exe youtube-dl.1 LATEST_VERSION
 
 PREFIX=/usr/local
 install: youtube-dl youtube-dl.1 youtube-dl.bash-completion
@@ -6,19 +10,21 @@ install: youtube-dl youtube-dl.1 youtube-dl.bash-completion
        install -m 644 --owner root --group root youtube-dl.1 $(PREFIX)/man/man1
        install -m 644 --owner root --group root youtube-dl.bash-completion /etc/bash_completion.d/youtube-dl
 
-.PHONY: all install
+.PHONY: all clean install README.md youtube-dl.bash-completion
+# TODO un-phony README.md and youtube-dl.bash_completion by reading from .in files and generating from them
 
 youtube-dl: youtube_dl/*.py
        zip --quiet --junk-paths youtube-dl youtube_dl/*.py
        echo '#!/usr/bin/env python' > youtube-dl
        cat youtube-dl.zip >> youtube-dl
        rm youtube-dl.zip
+       chmod a+x youtube-dl
 
 youtube-dl.exe: youtube_dl/*.py
        bash devscripts/wine-py2exe.sh build_exe.py
 
-README.md: youtube-dl
-       @options=$$(COLUMNS=80 ./youtube-dl --help | sed -e '1,/.*General Options.*/ d' -e 's/^\W\{2\}\(\w\)/## \1/') && \
+README.md: youtube_dl/*.py
+       @options=$$(COLUMNS=80 python -m youtube_dl --help | sed -e '1,/.*General Options.*/ d' -e 's/^\W\{2\}\(\w\)/## \1/') && \
                header=$$(sed -e '/.*# OPTIONS/,$$ d' README.md) && \
                footer=$$(sed -e '1,/.*# FAQ/ d' README.md) && \
                echo "$${header}" > README.md && \
@@ -37,5 +43,13 @@ youtube-dl.bash-completion: README.md
                content=`sed "s/opts=\"[^\"]*\"/opts=\"$${options}\"/g" youtube-dl.bash-completion` && \
                echo "$${content}" > youtube-dl.bash-completion
 
-LATEST_VERSION: youtube-dl
-       ./youtube-dl --version > LATEST_VERSION
+LATEST_VERSION: youtube_dl/__init__.py
+       python -m youtube_dl --version > LATEST_VERSION
+
+test:
+       nosetests2 test/*
+
+clean:
+       rm **/*.pyc
+
+.PHONY: default compile update update-latest update-readme test clean