IntenseWebs/sphinx
3
0
Fork 0
mirror of https://github.com/sphinx-doc/sphinx.git synced 2025-02-25 18:55:22 -06:00
Code Issues Packages Projects Releases Wiki Activity

Cache the result of `_file_checksum()`

Browse Source
This commit is contained in:
Adam Turner 2024-10-19 21:37:38 +01:00
parent 3c6ccd38c9
commit 10f6b6bb09
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
sphinx/builders/html/_assets.py
View File

@ -3,6 +3,7 @@ from __future__ import annotations
import os
import warnings
import zlib
from functools import cache
from typing import TYPE_CHECKING, Any, NoReturn
from sphinx.deprecation import RemovedInSphinx90Warning
@ -172,9 +173,14 @@ def _file_checksum(outdir: Path, filename: str | os.PathLike[str]) -> str:
if '?' in filename:
msg = f'Local asset file paths must not contain query strings: {filename!r}'
raise ThemeError(msg)
return _file_checksum_inner(outdir.joinpath(filename).resolve())
@cache
def _file_checksum_inner(file: Path) -> str:
try:
# Remove all carriage returns to avoid checksum differences
content = outdir.joinpath(filename).read_bytes().translate(None, b'\r')
content = file.read_bytes().translate(None, b'\r')
except FileNotFoundError:
return ''
if not content: