openstack
/
osprofiler
Code Issues Proposed changes

Don't set html_last_updated_fmt without git 

Using a timestamp as value for "html_last_updated_fmt" isn't good
for the reproducible builds initiative from different downstream
distributions because the build result always differs due to the
timestamp in the documentation.
Also when not setting "html_last_updated_fmt", the value can be set
via command line when running "sphinx-build -D".

Change-Id: Ia23451e6b53f6f83dd22e069ef42a4a067ec5881
This commit is contained in:
Thomas Bechtold 2016-04-13 09:22:10 +02:00
parent 9d410faa31
commit e8079b2ec3
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
doc/source/conf.py
View File

@ -15,7 +15,6 @@ import datetime
import os
import subprocess
import sys
import time
import warnings
# If extensions (or modules to document with autodoc) are in another directory,
@ -138,9 +137,8 @@ try:
html_last_updated_fmt = subprocess.Popen(
git_cmd, stdout=subprocess.PIPE).communicate()[0]
except Exception:
warnings.warn('Cannot get last updated time from git repository, '
'using current time.')
html_last_updated_fmt = time.ctime()
warnings.warn('Cannot get last updated time from git repository. '
'Not setting "html_last_updated_fmt".')
# If true, SmartyPants will be used to convert quotes and dashes to
# typographically correct entities.