blurb: don't overwrite output when not necessary

dmerejkowsky · dmerejkowsky · commit cf225d674a4e · 2021-05-15T15:31:35.000+02:00
This speeds up doc generation.

With this change, blurb won't update the NEWS file unnecessarily, which
means Sphinx will then use a cache instead of parsing it again.
diff --git a/blurb/blurb.py b/blurb/blurb.py
@@ -49,10 +49,12 @@
 from collections import OrderedDict
 import glob
 import hashlib
+import io
 import inspect
 import itertools
 import math
 import os
+from pathlib import Path
 import re
 import shlex
 import shutil
@@ -1068,9 +1070,11 @@ def merge(output=None, *, forced=False):
 
 
 def write_news(output, *, versions):
+    buff = io.StringIO()
+
     def print(*a, sep=" "):
         s = sep.join(str(x) for x in a)
-        return builtins.print(s, file=news)
+        return builtins.print(s, file=buff)
 
     print ("""
 +++++++++++
@@ -1131,7 +1135,20 @@ def print(*a, sep=" "):
             print(text)
     print()
     print("**(For information about older versions, consult the HISTORY file.)**")
-    news.close()
+
+
+    new_contents = buff.getvalue()
+
+    # Only write in `output` if the contents are different
+    # This speeds up subsequent Sphinx builds
+    try:
+        previous_contents = Path(output).read_text(encoding="UTF-8")
+    except (FileNotFoundError, UnicodeError):
+        previous_contents = None
+    if new_contents != previous_contents:
+        Path(output).write_text(new_contents, encoding="UTF-8")
+    else:
+        builtins.print(output, "is already up to date")
 
 
 git_add_files = []
