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

archlinux-latest-standard: System maxima does not work #36028

Closed
mkoeppe opened this issue Aug 3, 2023 · 8 comments · Fixed by #36391
Closed

archlinux-latest-standard: System maxima does not work #36028

mkoeppe opened this issue Aug 3, 2023 · 8 comments · Fixed by #36391

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Aug 3, 2023

Building the documentation fails with this:

https://github.com/sagemath/sage/actions/runs/5706565894/job/15465342057#step:8:2303

  [sagemath_doc_html-none]   ;;; Warning: Maxima is unable to set up the help system.
  [sagemath_doc_html-none]   (Details: CL-INFO::LOAD-PRIMARY-INDEX: Filesystem error with pathname #P"/usr/share/info/maxima-index.lisp".
  [sagemath_doc_html-none]   Either
  [sagemath_doc_html-none]    1) the file does not exist, or
  [sagemath_doc_html-none]    2) we are not allowed to access the file, or
  [sagemath_doc_html-none]    3) the pathname points to a broken symbolic link.)
  [sagemath_doc_html-none]   

@dimpase @antonio-rojas @orlitzky

@antonio-rojas
Copy link
Contributor

antonio-rojas commented Aug 3, 2023

Arch Linux docker images don't intall files in /usr/share/{man,info} by default for space saving reasons.

The line NoExtract = usr/share/man/* usr/share/info/* needs to be removed from /etc/pacman.conf in the docker image for this to work.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 3, 2023

OK, this means that we need to improve the spkg-configure.m4 for maxima

@dimpase
Copy link
Member

dimpase commented Aug 23, 2023

do we have ability to patch Docker images we use for CI? Surely, we can run an appropriate sed incantation before each call to pacman in build/bin/write-dockerfile.sh, but it's kinda ugly...

@orlitzky
Copy link
Contributor

What is failing? Sphinx is as helpful as ever.

@dimpase
Copy link
Member

dimpase commented Aug 23, 2023

on Archlinux's default Docker image, maxima docs are not installed by default, leading to a broken (for Sage's purposes) install of Maxima

@orlitzky
Copy link
Contributor

But why do we need the maxima docs to build what looks like the plotting documentation?

@dimpase
Copy link
Member

dimpase commented Aug 24, 2023

it does not seem to to be related to plotting. If I try to reproduce this locally on a Debian system, by removing the lisp file in question, I see

[sagemath_doc_html-none] sage --docbuild --no-pdf-links reference/combinat inventory --no-prune-empty-dirs
[sagemath_doc_html-none] [combinat ] building [inventory]: targets for 200 source files that are out of date
[sagemath_doc_html-none] [combinat ] updating environment: [config changed ('version')] 374 added, 0 changed, 1 removed
[sagemath_doc_html-none] [combinat ] /home/dimpase/work/software/sage/local/var/lib/sage/venv-python3.9/lib/python3.9/site-packages/sphinx/util/inspect.py:376: DeprecationWarning: this class is deprecated, do not use
[sagemath_doc_html-none] [combinat ] See https://github.com/sagemath/sage/issues/31545 for details.
[sagemath_doc_html-none] [combinat ]   return getattr(obj, name, *defargs)
[sagemath_doc_html-none] ;;; Warning: Maxima is unable to set up the help system.
[sagemath_doc_html-none] (Details: CL-INFO::LOAD-PRIMARY-INDEX: Filesystem error with pathname #P"/home/dimpase/work/software/sage/local/share/info/maxima-index.lisp".
[sagemath_doc_html-none] Either
[sagemath_doc_html-none]  1) the file does not exist, or
...

that is, sphinx is calling getattr() on a Sage object which insists on Maxima having help installed.

dimpase added a commit to dimpase/sage that referenced this issue Oct 3, 2023
This is needed for e.g. building Sage docs, cf
e.g. Sage's sagemath#36028. Some distos mix this up.

This fix was requested there.
@dimpase
Copy link
Member

dimpase commented Oct 3, 2023

OK, this means that we need to improve the spkg-configure.m4 for maxima

done in #36391

mkoeppe pushed a commit to dimpase/sage that referenced this issue Oct 8, 2023
This is needed for e.g. building Sage docs, cf
e.g. Sage's sagemath#36028. Some distos mix this up.

This fix was requested there.
@vbraun vbraun closed this as completed in d0073b2 Oct 14, 2023
@mkoeppe mkoeppe added this to the sage-10.2 milestone Oct 14, 2023
mkoeppe added a commit to mkoeppe/sage that referenced this issue Dec 25, 2023
mkoeppe added a commit to mkoeppe/sage that referenced this issue Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants