Closed giovannipizzi closed 2 years ago
Maybe @bosonie you have some idea (I guess SISL is a dependency of aiida-siesta?) Seems it cannot build: error: extension 'sisl.io.siesta._siesta' has Fortran sources but no Fortran compiler found
, probably we just need to instruct RTD to install Fortran (or have a way to install sisl without compiling, that's going to be even faster?)
Also, when did this change?
(The first PR where it fails is #205, but most probably it's due to a dependency change in some dependency, between 3 and 5 months ago (the two builds that resp. failed and succeeded))
We had already encountered this problem before with sisl
requiring a fortran compiler to be built, which I think is not ideal for a Python package, but apparently it is not easy to change for them or at least they don't want to: https://github.com/zerothi/sisl/issues/348
Looking at the docs of RTD, it doesn't seem that it is possible to install custom system requirements. Here they say that you cannot even install C-libraries, which is typically needed for Python code containing C-extensions. I take that to mean that installing a Fortran compiler is definitely not supported.
Maybe we can use this Sphinx feature though: https://www.sphinx-doc.org/en/master/usage/extensions/autodoc.html#confval-autodoc_mock_imports
Not sure if mocking the imports for either sisl
or aiida-siesta
will prevent the setup from trying to build sisl
.
The mocking feature seems nice! I'd be confident that sisl
is not required for the docs, but @bosonie can confirm (and it seems easy to setup and test?)
See e.g. the bottom of the homepage, where in the code there is the correct citation, but it's not visible in the HTML rendered version.
If I login in RTD, I think the most recent error log is this, if it helps: