openstack
/
openstack-health
Code Issues Proposed changes

Replace deprecated library function os.popen() with subprocess 

os.popen() is deprecated since version 2.6. Resolved with use of
subprocess module.

Change-Id: I409af47bb09cd5ba48e0e32a424b35d1f61323e9
Closes-Bug: #1529836
This commit is contained in:
Harshada Mangesh Kakad 2016-01-14 05:14:24 -08:00
parent f0bbb34ebe
commit 663c2f4d5a
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
doc/source/conf.py
View File

@ -1,3 +1,4 @@
import subprocess
import sys
import os
@ -109,8 +110,10 @@ html_static_path = ['_static']
# If not '', a 'Last updated on:' timestamp is inserted at every page bottom,
# using the given strftime format.
git_cmd = "git log --pretty=format:'%ad, commit %h' --date=local -n1"
html_last_updated_fmt = os.popen(git_cmd).read()
git_cmd = ["git", "log", "--pretty=format:'%ad, commit %h'", "--date=local",
"-n1"]
html_last_updated_fmt = subprocess.Popen(
git_cmd, stdout=subprocess.PIPE).communicate()[0]
# If true, SmartyPants will be used to convert quotes and dashes to
# typographically correct entities.