[Debichem-devel] Bug#1107280: pymatgen: FTBFS: Tries to access Internet during build

[Santiago Vila]

Package: src:pymatgen
Version: 2025.2.18+dfsg1-4
Severity: serious
Tags: ftbfs trixie sid

Dear maintainer:

During a rebuild of all packages in unstable, your package failed to build:

--------------------------------------------------------------------------------
[...]
self = <urllib3.connectionpool.HTTPSConnectionPool object at 0x7fce1d0e9ba0>
err = TimeoutError('The read operation timed out')
url = '/wiki/api.php?action=query&list=categorymembers&cmtitle=Category:INCAR_tag&cmlimit=500&format=json'
timeout_value = 60

     def _raise_timeout(
         self,
         err: BaseSSLError | OSError | SocketTimeout,
         url: str,
         timeout_value: _TYPE_TIMEOUT | None,
     ) -> None:
         """Is the error actually a timeout? Will raise a ReadTimeout or pass"""
     
         if isinstance(err, SocketTimeout):
>           raise ReadTimeoutError(
                 self, url, f"Read timed out. (read timeout={timeout_value})"
             ) from err
E           urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host='www.vasp.at', port=443): Read timed out. (read timeout=60)
--------------------------------------------------------------------------------

The reason is in tests/io/vasp/test_help.py which reads like this:


try:
     website_down = requests.get("https://www.vasp.at", timeout=5).status_code != 200
except (requests.exceptions.ConnectionError, requests.exceptions.ReadTimeout):
     website_down = True


I guess that for the package build as such, we could do this instead of the
above four lines:

website_down = True

and the remaining code would not need to be touched.


But I see in #1103427 that some network tests are deliberately run in CI
so I'm not sure if the above simple fix would be compatible with running
the network tests in CI, so I leave it to you.

Thanks.