Say why we allow override of build timestamp

It's for reproducible builds.

Also, fix Python formatting.

CHANGES

@@ -3,6 +3,8 @@ Revision history for SPIRV-Tools
 v2016.7-dev 2017-01-06
  - Add build target spirv-tools-vimsyntax to generate spvasm.vim, a SPIR-V
    assembly syntax file for Vim.
+ - Version string: Allow overriding of wall clock timestamp with contents
+   of environment variable SOURCE_DATE_EPOCH.
  - Fixes:
    #508: Support compilation under CYGWIN
    #517: Fix validation when continue (or case) contstruct is also the head of a

utils/update_build_version.py

@@ -110,9 +110,15 @@ def describe(directory):
             return command_output(
                 ['git', 'rev-parse', 'HEAD'], directory).rstrip().decode()
         except:
-            return 'unknown hash, {}'.format(datetime.date.fromtimestamp(
+            # This is the fallback case where git gives us no information,
-                int(os.environ.get('SOURCE_DATE_EPOCH', time.time()))
+            # e.g. because the source tree might not be in a git tree.
-                ).isoformat())
+            # In this case, usually use a timestamp.  However, to ensure
+            # reproducible builds, allow the builder to override the wall
+            # clock time with enviornment variable SOURCE_DATE_EPOCH
+            # containing a (presumably) fixed timestamp.
+            timestamp = int(os.environ.get('SOURCE_DATE_EPOCH', time.time()))
+            formatted = datetime.date.fromtimestamp(timestamp).isoformat()
+            return 'unknown hash, {}'.format(formatted)
 
 
 def main():