Remove unused patch

1. Introduce --without websupport

 Sphinx (build)requires sphinxcontrib-websupport but that
 (build)requires sphinx. This bcond can be used to bootstrap this.

2. Introduce --without tests

 The tests bring a lot of additional dependencies that make
 bootstrapping harder. This allows to build without tests.

3. Build the docs with self

 The documentation was built with 'python', thus importing sphinx
 from python2's sitelib. Sphinx was installed because of the above
 mentioned buildrequirement of sphinxcontrib-websupport.
 Now current sphinx from PWD is used, also python3 is used when
 available, otherwise python2. Never just python.

.gitignore

@@ -13,3 +13,5 @@
 /Sphinx-1.6.6.tar.gz
 /Sphinx-1.7.1.tar.gz
 /Sphinx-1.7.2.tar.gz
+/Sphinx-1.7.5.tar.gz
+/Sphinx-1.7.6.tar.gz

fix-test_autodoc.patch

@@ -1,32 +0,0 @@
-diff --git a/sphinx/util/inspect.py b/sphinx/util/inspect.py
-index e8427fd84..492db8700 100644
---- a/sphinx/util/inspect.py
-+++ b/sphinx/util/inspect.py
-@@ -313,7 +313,10 @@ def __init__(self, subject, bound_method=False):
-             try:
-                 self.signature = inspect.signature(subject)
-             except IndexError:
--                if hasattr(subject, '_partialmethod'):  # partialmethod with no argument
-+                # Until python 3.6.4, cpython has been crashed on inspection for
-+                # partialmethods not having any arguments.
-+                # https://bugs.python.org/issue33009
-+                if hasattr(subject, '_partialmethod'):
-                     self.signature = None
-                     self.partialmethod_with_noargs = True
-                 else:
-diff --git a/tests/test_autodoc.py b/tests/test_autodoc.py
-index 9398edc34..ed6d050fa 100644
---- a/tests/test_autodoc.py
-+++ b/tests/test_autodoc.py
-@@ -952,7 +952,10 @@ def call_autodoc(objtype, name):
-         '      Update state of cell to *state*.',
-         '      ',
-     ]
--    if sys.version_info < (3, 5, 4):
-+    if (sys.version_info < (3, 5, 4) or
-+            (3, 6, 5) <= sys.version_info < (3, 7) or
-+            (3, 7, 0, 'beta', 3) <= sys.version_info):
-+        # TODO: this condition should be updated after 3.7-final release.
-         expected = '\n'.join(expected).replace(' -> None', '').split('\n')
- 
-     assert call_autodoc('class', 'target.partialmethod.Cell') == expected

python-sphinx.spec

@@ -11,6 +11,11 @@
 %bcond_without imagemagick_tests
 %endif
 
+# When bootstrapping sphinx, we don't yet have sphinxcontrib-websupport
+%bcond_without websupport
+# Also, we don't have all the tests requirements
+%bcond_without tests
+
 # Currently, python2 version is always the default: https://fedoraproject.org/wiki/Packaging:Python#Naming
 %if 1
 %global py3_default 0
@@ -21,8 +26,8 @@
 %global upstream_name Sphinx
 
 Name:       python-sphinx
-Version:    1.7.2
-Release:    2%{?dist}
+Version:    1.7.6
+Release:    1%{?dist}
 Epoch:      1
 Summary:    Python documentation generator
 
@@ -49,9 +54,9 @@ Source6:    default-sphinx-command.in
 # to fetch images, which is not possible in koji or mock.
 Patch0: xfail-test_latex_remote_images.patch
 
-# Fix test_autodoc failure with Python 3.6.5
-# Fixed upstream: https://github.com/sphinx-doc/sphinx/pull/4822
-patch1: fix-test_autodoc.patch
+# Allow extra themes to exist. We pull in python[23]-sphinx-theme-alabaster
+# which causes that test to fail.
+Patch1: sphinx-test_theming.diff
 
 BuildArch:     noarch
 BuildRequires: python2-devel >= 2.4
@@ -67,13 +72,15 @@ BuildRequires: python2-imagesize
 BuildRequires: python2-requests
 BuildRequires: python2-packaging
 BuildRequires: python2-typing
+%if %{with websupport}
 BuildRequires: python2-sphinxcontrib-websupport
+%endif
 BuildRequires: environment(modules)
 
 # for fixes
 BuildRequires: dos2unix
 
-# for testing
+%if %{with tests}
 BuildRequires: python2-nose
 BuildRequires: python2-pytest
 BuildRequires: python2-pytest-cov
@@ -82,7 +89,6 @@ BuildRequires: texinfo
 BuildRequires: graphviz
 BuildRequires: python2-sqlalchemy
 BuildRequires: python2-mock
-BuildRequires: python2-simplejson
 BuildRequires: python2-html5lib
 BuildRequires: python2-whoosh
 BuildRequires: python2-snowballstemmer
@@ -90,8 +96,6 @@ BuildRequires: python2-enum34
 %if %{with imagemagick_tests}
 BuildRequires: ImageMagick
 %endif
-# note: no Python3 xapian binding yet
-BuildRequires: python2-xapian
 BuildRequires: texlive-collection-fontsrecommended
 BuildRequires: texlive-collection-latex
 BuildRequires: texlive-dvipng
@@ -126,6 +130,7 @@ BuildRequires: tex(tabulary.sty)
 BuildRequires: tex(polyglossia.sty)
 BuildRequires: tex(ctablestack.sty)
 BuildRequires: tex(eu1enc.def)
+%endif
 
 %if 0%{?with_python3}
 BuildRequires: python3-devel
@@ -134,22 +139,25 @@ BuildRequires: python3-setuptools
 BuildRequires: python3-docutils
 BuildRequires: python3-jinja2
 BuildRequires: python3-pygments
-BuildRequires: python3-nose
-BuildRequires: python3-pytest
-BuildRequires: python3-pytest-cov
-BuildRequires: python3-sqlalchemy
-BuildRequires: python3-mock
-BuildRequires: python3-simplejson
-BuildRequires: python3-html5lib
-BuildRequires: python3-whoosh
-BuildRequires: python3-snowballstemmer
 BuildRequires: python3-six
 BuildRequires: python3-sphinx_rtd_theme
 BuildRequires: python3-sphinx-theme-alabaster
 BuildRequires: python3-packaging
 BuildRequires: python3-imagesize
 BuildRequires: python3-requests
+%if %{with websupport}
 BuildRequires: python3-sphinxcontrib-websupport
+%endif
+%if %{with tests}
+BuildRequires: python3-nose
+BuildRequires: python3-pytest
+BuildRequires: python3-pytest-cov
+BuildRequires: python3-sqlalchemy
+BuildRequires: python3-mock
+BuildRequires: python3-html5lib
+BuildRequires: python3-whoosh
+BuildRequires: python3-snowballstemmer
+%endif
 %endif # with_python3
 
 
@@ -198,7 +206,9 @@ Requires:      python2-imagesize
 Requires:      python2-requests
 Requires:      python2-packaging
 Requires:      python2-typing
+%if %{with websupport}
 Requires:      python2-sphinxcontrib-websupport
+%endif
 Requires: environment(modules)
 # Needed to get rid of the alternatives config installed in f24 and f25
 # versions of the package
@@ -310,8 +320,10 @@ Requires:      python3-sphinx-theme-alabaster
 Requires:      python3-imagesize
 Requires:      python3-requests
 Requires:      python3-six
-Requires:      python3-sphinxcontrib-websupport
 Requires:      python3-packaging
+%if %{with websupport}
+Requires:      python3-sphinxcontrib-websupport
+%endif
 Recommends:    graphviz
 Recommends:    ImageMagick
 Requires: environment(modules)
@@ -394,10 +406,14 @@ cp %{SOURCE5} .
 # fix line encoding of bundled jquery.js
 dos2unix -k ./sphinx/themes/basic/static/jquery.js
 
-%if ! %{with imagemagick_tests}
+%if %{without imagemagick_tests}
 rm tests/test_ext_imgconverter.py
 %endif
 
+%if %{without websupport}
+rm tests/test_websupport.py tests/test_api_translator.py
+%endif
+
 %if 0%{?with_python3}
 rm -rf %{py3dir}
 cp -a . %{py3dir}
@@ -409,9 +425,16 @@ cp -a . %{py3dir}
 %py3_build
 %endif # with_python3
 
+export PYTHONPATH=$PWD
 pushd doc
-make html
-make man
+%if 0%{?with_python3}
+export SPHINXBUILD="%{__python3} ../sphinx/cmd/build.py"
+%else
+export SPHINXBUILD="%{__python2} ../sphinx/cmd/build.py"
+%endif
+
+make html SPHINXBUILD="$SPHINXBUILD"
+make man SPHINXBUILD="$SPHINXBUILD"
 rm -rf _build/html/.buildinfo
 mv _build/html ..
 popd
@@ -520,10 +543,14 @@ done
   >> sphinx.lang
 
 
+%if %{with tests}
 %check
+export PYTHONPATH=$PWD
+
 # Currently, all linkcheck tests hit external websites.  Since network access
 # is disabled in koji, we have to disable these.
 rm tests/test_build_linkcheck.py
+
 # Igoring test_api_translator for now as it fails on Fedora.
 # The issue has been reported upstream: https://github.com/sphinx-doc/sphinx/issues/4710
 LANG=en_US.UTF-8 %{__python2} -m pytest -v --ignore=tests/test_api_translator.py
@@ -535,6 +562,7 @@ rm tests/test_build_linkcheck.py
 LANG=en_US.UTF-8 PYTHON=python3 make test
 popd
 %endif # with_python3
+%endif # with tests
 
 
 %files latex
@@ -590,6 +618,37 @@ popd
 
 
 %changelog
+* Mon Jul 23 2018 Marcel Plch <mplch@redhat.com> - 1:1.7.6-1
+- Update to 1.7.6
+
+* Fri Jul 13 2018 Miro Hrončok <mhroncok@redhat.com> - 1:1.7.5-6
+- Remove unused dependencies on xapian and simplejson
+
+* Thu Jul  5 2018 Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> - 1:1.7.5-5
+- Add patch to fix build if alabaster theme is installed
+- Add patch for #1589868
+
+* Mon Jul 02 2018 Miro Hrončok <mhroncok@redhat.com> - 1:1.7.5-4
+- Enable tests
+
+* Mon Jul 02 2018 Miro Hrončok <mhroncok@redhat.com> - 1:1.7.5-3
+- Enable websupport
+
+* Mon Jul 02 2018 Miro Hrončok <mhroncok@redhat.com> - 1:1.7.5-2
+- Rebuilt for Python 3.7
+
+* Tue Jun 19 2018 Charalampos Stratakis <cstratak@redhat.com> - 1:1.7.5-1
+- Update to 1.7.5 (bz#1570451)
+
+* Mon Jun 18 2018 Miro Hrončok <mhroncok@redhat.com> - 1:1.7.2-5
+- Rebuilt for Python 3.7
+
+* Fri Jun 15 2018 Miro Hrončok <mhroncok@redhat.com> - 1:1.7.2-4
+- Bootstrap for Python 3.7
+
+* Thu Jun 14 2018 Miro Hrončok <mhroncok@redhat.com> - 1:1.7.2-3
+- Bootstrap for Python 3.7
+
 * Wed Apr 11 2018 Petr Viktorin <pviktori@redhat.com> - 1:1.7.2-2
 - Conditionalize the ImageMagick build dependency & tests
 

sources

@@ -1 +1 @@
-SHA512 (Sphinx-1.7.2.tar.gz) = 6cb055d969227c727e1cdd8865ddef700d72217e0ef3700383449921e9310f7ed6266d62cf1928ad64372f93468d457c584fac34f6583714cc8a9a3627785ac3
+SHA512 (Sphinx-1.7.6.tar.gz) = 505c4e92c1a09c48a50d8bd0ec86089bd3f0f5352302ae330e2fba6e61316f40c6ec3726e2cfaeb54de0818127a26ef5364ac92bcf254283022c38c4e95d419c

sphinx-test_theming.diff

@@ -0,0 +1,12 @@
+diff -ru Sphinx-1.7.6/tests/test_theming.py Sphinx-1.7.6_patched/tests/test_theming.py
+--- Sphinx-1.7.6/tests/test_theming.py	2018-07-16 11:24:40.000000000 +0200
++++ Sphinx-1.7.6_patched/tests/test_theming.py	2018-07-20 15:17:35.049263077 +0200
+@@ -31,7 +31,7 @@
+         themes.append('alabaster')
+ 
+     # test Theme class API
+-    assert set(app.html_themes.keys()) == set(themes)
++    assert set(app.html_themes.keys()) >= set(themes)
+     assert app.html_themes['test-theme'] == app.srcdir / 'test_theme' / 'test-theme'
+     assert app.html_themes['ziptheme'] == app.srcdir / 'ziptheme.zip'
+     assert app.html_themes['staticfiles'] == app.srcdir / 'test_theme' / 'staticfiles'