Fixes to build process.

ninoNinkovic · Jul 11, 2014 · b1a3b87 · b1a3b87
1 parent 68966f4
commit b1a3b87
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 5 deletions.
diff --git a/.gitignore b/.gitignore
@@ -2,3 +2,4 @@
 _in
 dist
 MANIFEST
+build
diff --git a/Makefile b/Makefile
@@ -1,16 +1,22 @@
 SHELL := /bin/bash
-.PHONY : all package install uninstall
+.PHONY : all package install uninstall test
 
-all :
+all : dist/ALE-0.1dev.tar.gz
+
+clean :
+	rm -rf *.pyc dist MANIFEST build 
 
 package : ale.py MANIFEST.in README.md setup.py
 	python setup.py sdist
 
 dist/ALE-0.1dev.tar.gz : package
 
-install : dist/ALE-0.1dev.tar.gz
+install :
 	sudo pip install dist/ALE-0.1dev.tar.gz
 
 uninstall : dist/ALE-0.1dev.tar.gz
 	yes | sudo pip uninstall ale
 
+test : 
+	cd test && ./read_test.py
+
diff --git a/README.md b/README.md
@@ -1,3 +1,7 @@
 # Python ALE (python-ale)
 
-A reader for Avid ALE files.
+A reader for Avid ALE files.
+
+## Installation
+
+    make && make install
diff --git a/ale.py b/ale.py
@@ -136,7 +136,7 @@ def _read_heading(self, line):
             raise ValueError("Unknown heading: '%s'.", key)
         heading = headings[key]
 
-        if heading["options"]:
+        if heading.get("options", None):
             for option in heading["options"]:
                 if value.lower() == option.lower():
                     setattr(self, heading['name'], option)
-Original file line number
+Diff line change
@@ Expand Up / @@ -2,3 +2,4 @@ @@
     _in
     dist
     MANIFEST
+    build