Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Smaller/wrong search index with sphinx-build -j auto #11163

Closed
rogerbinns opened this issue Jan 30, 2023 · 2 comments · Fixed by #11192
Closed

Smaller/wrong search index with sphinx-build -j auto #11163

rogerbinns opened this issue Jan 30, 2023 · 2 comments · Fixed by #11192

Comments

@rogerbinns
Copy link

Describe the bug

Using the latest version and building with -j auto results in a considerably smaller searchindex,js and as a result most searches returning nothing. If I leave out -j then the searchindex is considerably larger and searches work as expected.

Some quick testing showed that Sphinx 6.0.0 did not have this problem while Sphinx 6.1.0 onwards does.

How to Reproduce

This fails:

sphinx-build -j auto  -n -b html -d build/doctrees -q -E . build/html

This works:

sphinx-build -n -b html -d build/doctrees -q -E . build/html

Environment Information

Platform:              linux; (Linux-5.19.0-29-generic-x86_64-with-glibc2.36)
Python version:        3.10.7 (main, Nov 24 2022, 19:45:47) [GCC 12.2.0])
Python implementation: CPython
Sphinx version:        6.1.3
Docutils version:      0.19
Jinja2 version:        3.0.3
Pygments version:      2.14.0

Sphinx extensions

No response

Additional context

Sphinx conf.py (almost empty!) and other files are at https://github.com/rogerbinns/apsw/tree/master/doc

@phlax
Copy link

phlax commented Jan 31, 2023

im also hitting this and can confirm that downgrading to 6.0.1 resolves

jbms added a commit to jbms/sphinx that referenced this issue Feb 12, 2023
The image-related changes were already reverted in
2a7c40d, but the search index changes
also need to be reverted.

It would be nice to support parallel search index building, but the
necessary support for merging the search indices produced by each
process needs to be added first.

Fixes sphinx-doc#11163.
@jbms
Copy link
Contributor

jbms commented Feb 12, 2023

I created a fix here: #11192

jbms added a commit to jbms/sphinx that referenced this issue Feb 12, 2023
The image-related changes were already reverted in
2a7c40d, but the search index changes
also need to be reverted.

It would be nice to support parallel search index building, but the
necessary support for merging the search indices produced by each
process needs to be added first.

Fixes sphinx-doc#11163.
copybara-service bot pushed a commit to google/tensorstore that referenced this issue Feb 13, 2023
Downgrades Sphinx to avoid sphinx-doc/sphinx#11163.

PiperOrigin-RevId: 509322185
Change-Id: Ic1e83d3e689762f318679a09f716466706f74289
rouault added a commit to rouault/gdal that referenced this issue Feb 16, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants