Skip to content

Commit

Permalink
Merge pull request #1268 from ldrumm/subprocess-pipes
Browse files Browse the repository at this point in the history
[gen] subprocess.run result pipes are not files
  • Loading branch information
kbenzie authored Jan 23, 2024
2 parents 7c58060 + 9c6f84e commit 6fb1e54
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions scripts/generate_docs.py
Original file line number Diff line number Diff line change
Expand Up @@ -262,7 +262,7 @@ def generate_html(dstpath):
if result.returncode != 0:
print("sphinx-build returned non-zero error code.")
print("--- output ---")
print(result.stderr.read().decode())
print(result.stderr.decode())
raise Exception("Failed to generate html documentation.")

"""
Expand All @@ -277,7 +277,7 @@ def generate_pdf(dstpath):
if result.returncode != 0:
print("sphinx-build returned non-zero error code.")
print("--- output ---")
print(result.stderr.read().decode())
print(result.stderr.decode())
raise Exception("Failed to generate pdf documentation.")

"""
Expand Down

0 comments on commit 6fb1e54

Please sign in to comment.