Move mathjax detection to docbuild meson file

sagemath · tobiasdiez · Jul 9, 2025 · Jul 9, 2025 · Jul 11, 2025 · Jul 11, 2025
commit 9dd0ab8eb37d6221bb2534befead4f5825915c5a
diff --git a/meson.build b/meson.build
@@ -218,37 +218,4 @@ run_command(create_files_command, check: true)
 
 root = meson.current_source_dir()
 
-# MathJax detection logic
-# Check for system-wide MathJax installation
-mathjax_dir = ''
-mathjax_found = false
-
-# Check common system locations for MathJax
-foreach dir : ['/usr/share/mathjax', '/usr/local/share/mathjax']
-  if not mathjax_found and fs.is_file(dir / 'tex-chtml.js')
-    mathjax_dir = dir
-    mathjax_found = true
-    message('Found MathJax at: ' + dir)
-  endif
-endforeach
-
-# If not found, use the bundled/prefix location
-if not mathjax_found
-  mathjax_dir = get_option('prefix') / 'share' / 'mathjax' / 'mathjax'
-  message('Using bundled MathJax location: ' + mathjax_dir)
-endif
-
-# Generate the configuration template
-mathjax_config_data = configuration_data()
-mathjax_config_data.set('MATHJAX_DIR', mathjax_dir)
-
-# Create config file in the build directory
-configure_file(
-  input: 'src/sage_docbuild/mathjax_config.py.in',
-  output: 'mathjax_config.py',
-  configuration: mathjax_config_data,
-  install: true,
-  install_dir: get_option('prefix') / 'lib' / 'python' + py.language_version() / 'site-packages' / 'sage_docbuild'
-)
-
 subdir('src')
diff --git a/src/meson.build b/src/meson.build
@@ -238,4 +238,5 @@ src = meson.current_source_dir()
 
 # Submodules
 subdir('sage')
+subdir('sage_docbuild')
 subdir('doc')
diff --git a/src/sage_docbuild/conf.py b/src/sage_docbuild/conf.py
@@ -34,13 +34,7 @@
 from sage.misc.sagedoc import extlinks as extlinks  # noqa: PLC0414
 from sage.misc.sagedoc_conf import *  # Load configuration shared with sage.misc.sphinxify
 
-# Import MathJax configuration from the build system
-try:
-    from sage_docbuild.mathjax_config import MATHJAX_DIR
-except ImportError:
-    # Fallback to default location if config file is not available
-    import os
-    MATHJAX_DIR = os.path.join(os.environ.get('SAGE_LOCAL', '/usr/local'), 'share', 'mathjax')
+from .config import MATHJAX_PATH
 
 # ---------------------
 # General configuration
@@ -558,7 +552,7 @@ def linkcode_resolve(domain, info):
 if os.environ.get('SAGE_USE_CDNS', 'no') == 'yes':
     mathjax_path = "https://cdn.jsdelivr.net/npm/mathjax@3/es5/tex-chtml.js"
 else:
-    mathjax_path = os.path.join(MATHJAX_DIR, 'tex-chtml.js')
+    mathjax_path = MATHJAX_PATH
 
 # A list of glob-style patterns that should be excluded when looking for source
 # files. They are matched against the source file names relative to the

diff --git a/src/sage_docbuild/config.py.in b/src/sage_docbuild/config.py.in
@@ -0,0 +1,6 @@
+"""
+Documentation configuration generated by meson
+"""
+
+# MathJax path
+MATHJAX_PATH: str = '@MATHJAX_PATH@'
diff --git a/src/sage_docbuild/mathjax_config.py.in b/src/sage_docbuild/mathjax_config.py.in
diff --git a/src/sage_docbuild/meson.build b/src/sage_docbuild/meson.build
@@ -0,0 +1,21 @@
+# Check for system-wide MathJax installation
+mathjax_path = ''
+mathjax_found = false
+foreach dir : ['/usr/share/mathjax', '/usr/local/share/mathjax']
+  if not mathjax_found and fs.is_file(dir / 'tex-chtml.js')
+    mathjax_path = dir / 'tex-chtml.js'
+    mathjax_found = true
+    message('Found MathJax at: ' + dir)
+  endif
+endforeach
+if not mathjax_found
+  mathjax_path = 'https://cdn.jsdelivr.net/npm/mathjax@3/es5/tex-chtml.js'
+  message('Using CDN MathJax: ' + mathjax_path)
+endif
+docs_config_data = configuration_data()
+docs_config_data.set('MATHJAX_PATH', mathjax_path)
+configure_file(
+  input: 'config.py.in',
+  output: 'config.py',
+  configuration: docs_config_data,
+)