Skip to content

Commit

Permalink
Merge pull request #9826 from tk0miya/9820_reduce_distutils_usage
Browse files Browse the repository at this point in the history
Replace distutils.versions.LooseVersion by packaging.version.Version
  • Loading branch information
tk0miya committed Nov 8, 2021
2 parents 8e35049 + ee1cae8 commit 6c6cc8a
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 9 deletions.
4 changes: 2 additions & 2 deletions sphinx/highlighting.py
Expand Up @@ -8,11 +8,11 @@
:license: BSD, see LICENSE for details.
"""

from distutils.version import LooseVersion
from functools import partial
from importlib import import_module
from typing import Any, Dict

from packaging import version
from pygments import __version__ as pygmentsversion
from pygments import highlight
from pygments.filters import ErrorToken
Expand Down Expand Up @@ -64,7 +64,7 @@
{\let\fcolorbox\spx@fixpyg@fcolorbox\PYG@do{#2}}}
\makeatother
'''
if tuple(LooseVersion(pygmentsversion).version) <= (2, 7, 4):
if version.parse(pygmentsversion).release <= (2, 7, 4):
_LATEX_ADD_STYLES += _LATEX_ADD_STYLES_FIXPYG


Expand Down
4 changes: 2 additions & 2 deletions sphinx/util/docutils.py
Expand Up @@ -12,7 +12,6 @@
import re
from contextlib import contextmanager
from copy import copy
from distutils.version import LooseVersion
from os import path
from types import ModuleType
from typing import (IO, TYPE_CHECKING, Any, Callable, Dict, Generator, List, Optional, Set,
Expand All @@ -26,6 +25,7 @@
from docutils.parsers.rst.states import Inliner
from docutils.statemachine import State, StateMachine, StringList
from docutils.utils import Reporter, unescape
from packaging import version

from sphinx.errors import SphinxError
from sphinx.locale import _
Expand All @@ -41,7 +41,7 @@
from sphinx.environment import BuildEnvironment


__version_info__ = tuple(LooseVersion(docutils.__version__).version)
__version_info__ = version.parse(docutils.__version__).release
additional_nodes: Set[Type[Element]] = set()


Expand Down
11 changes: 6 additions & 5 deletions tests/test_build_html.py
Expand Up @@ -10,13 +10,13 @@

import os
import re
from distutils.version import LooseVersion
from itertools import chain, cycle
from unittest.mock import ANY, call, patch

import pygments
import pytest
from html5lib import HTMLParser
from packaging import version

from sphinx.builders.html import validate_html_extra_path, validate_html_static_path
from sphinx.errors import ConfigError
Expand All @@ -30,6 +30,9 @@
FIGURE_CAPTION = ".//figure/figcaption/p"


PYGMENTS_VERSION = version.parse(pygments.__version__).release


ENV_WARNINGS = """\
%(root)s/autodoc_fodder.py:docstring of autodoc_fodder.MarkupError:\\d+: \
WARNING: Explicit markup ends without a blank line; unexpected unindent.
Expand Down Expand Up @@ -1576,8 +1579,7 @@ def test_html_codeblock_linenos_style_table(app):
app.build()
content = (app.outdir / 'index.html').read_text()

pygments_version = tuple(LooseVersion(pygments.__version__).version)
if pygments_version >= (2, 8):
if PYGMENTS_VERSION >= (2, 8):
assert ('<div class="linenodiv"><pre><span class="normal">1</span>\n'
'<span class="normal">2</span>\n'
'<span class="normal">3</span>\n'
Expand All @@ -1592,8 +1594,7 @@ def test_html_codeblock_linenos_style_inline(app):
app.build()
content = (app.outdir / 'index.html').read_text()

pygments_version = tuple(LooseVersion(pygments.__version__).version)
if pygments_version > (2, 7):
if PYGMENTS_VERSION > (2, 7):
assert '<span class="linenos">1</span>' in content
else:
assert '<span class="lineno">1 </span>' in content
Expand Down

0 comments on commit 6c6cc8a

Please sign in to comment.