starting to handle errors correclty

Author: nir0s

README.md

@@ -6,4 +6,36 @@ python-packer
 * Version [![PypI](http://img.shields.io/pypi/v/python-packer.svg)](http://img.shields.io/pypi/v/python-packer.svg)
 
 
-A Python interface for executing [packer.io](http://www.packer.io)
+A Python client for [packer.io](http://www.packer.io)
+
+## Installation
+
+You must have Packer installed prior to using this client (DUH!)
+
+```shell
+ pip install python-packer
+ # or, for dev:
+ pip install https://github.com/nir0s/python-packer/archive/master.tar.gz
+```
+
+## Usage Example
+
+### Executing a build
+
+```python
+import packer
+
+packerfile = 'packer/tests/resources/packerfile.json'
+packer_exec_path = '/usr/bin/packer'
+exc = []
+only = []
+vars = {"variable1": "y", "variable2": "value"}
+vars_file = 'path/to/vars/file'
+
+p = packer.Packer(packerfile, exc=exc, only=only, vars=vars,
+                  vars_file=vars_file, exec_path=packer_exec_path)
+# print(p.validate(syntax_only=True))
+print(p.build())
+result = p.inspect()
+print result.parsed_output
+```

packer/__init__.py

@@ -62,7 +62,13 @@ def validate(self, syntax_only=False):
             validator = validator.bake('-syntax-only')
         validator = self._append_base_arguments(validator)
         validator = validator.bake(self.packerfile)
-        return validator()
+        try:
+            validated = validator()
+        except:
+            print 'HASDHASD'
+        validated.succeeded = True if validated.exit_code == 0 else False
+        validated.failed = not validated.succeeded
+        return validated
 
     def build(self, parallel=True, debug=False, force=False):
         """Executes a Packer build (`packer build`)
@@ -108,9 +114,10 @@ def inspect(self):
         """
         inspector = self.packer.inspect
         inspector = inspector.bake('-machine-readable', self.packerfile)
-        result = inspector()
-        result.parsed_output = self._parse_inspection_output(result.stdout)
-        return result
+        inspected = inspector()
+        inspected.parsed_output = self._parse_inspection_output(
+            inspected.stdout)
+        return inspected
 
     def _parse_inspection_output(self, output):
         """Parses the machine-readable output `packer inspect` provides.

tester.py

@@ -1,8 +1,6 @@
 import packer
-import os
-
-packerfile = os.path.expanduser('packer/tests/resources/packerfile.json')  # NOQA
 
+packerfile = 'packer/tests/resources/packerfile.json'
 # exc = ['z', 't']
 exc = []
 # only = ['x', 'y']
@@ -14,7 +12,7 @@
 p = packer.Packer(packerfile, exc=exc, only=only, vars=vars)
 # print(p.version())
 
-# print(p.validate(syntax_only=True))
+print(p.validate(syntax_only=False))
 # print(p.build())
-result = p.inspect()
-print result.parsed_output
+# result = p.inspect()
+# print result.parsed_output