ValueError: invalid inventory header (not compressed)

AstraLuma commented 2 years ago

Describe the bug

I have found inventories in the wild that sphinx does not like but otherwise appear to be valid.

I'm currently not sure the providence of these inventories.

How to Reproduce

Intersphinx to any of these:

https://array-split.readthedocs.io/en/latest/objects.inv
https://django-simple-notifications.readthedocs.io/en/latest/objects.inv
https://urecord.readthedocs.io/en/latest/objects.inv
https://docs.lavasoftware.org/lavacli/objects.inv
https://mailbot.readthedocs.io/en/latest/objects.inv

Expected behavior

Parse and process these

Your project

https://github.com/AstraLuma/catfind/

Screenshots

No response

OS

Linux

Python version

3.9

Sphinx version

-

Sphinx extensions

No response

Extra tools

No response

Additional context

I found this by integrating Sphinx's inventory code and working on my own inventory tools, not by using Sphinx directly.

While I did vendor the Sphinx inventory code, I strongly believe this bug exists in both copies.

The bug in my software is https://github.com/AstraLuma/catfind/issues/15

tk0miya commented 2 years ago

It seems these inventories contains whitespaces inside its version info. It was reported in #3488 and fixed at v1.5.4. So it will be resolved if these projects will rebuild document.

AstraLuma commented 2 years ago

"Wait for them to not do the thing" isn't really a solution for "our parser is unnecessarily restrictive compared to what appears in the wild". Especially in the face that the already built inventories will always be attached to old versions of the docs (for sites like Read The Docs that retain old versions).

sphinx-doc / sphinx