[bug#43185] [PATCH 0/1] gnu: emacs-auctex: Configure executable paths.

[bug#43185] [PATCH 0/1] gnu: emacs-auctex: Configure executable paths.

[Arun Isaac]

This patch configures variables in emacs-auctex to use absolute store paths to
ghostscript and texlive commands. This means we have to add ghostscript to the
inputs. This, of course, greatly increases the closure size of
emacs-auctex. But, ghostscript is required for auctex to generate in buffer
previews. Generating in buffer previews is an important feature of auctex and
users would expect that to work out of the box without having to manually
install ghostscript into their profile.

Arun Isaac (1):
  gnu: emacs-auctex: Configure executable paths.

 gnu/packages/emacs-xyz.scm | 19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)

-- 
2.28.0

[bug#43185] [PATCH] gnu: emacs-auctex: Configure executable paths.

[Arun Isaac]

* gnu/packages/emacs-xyz.scm (emacs-auctex): Configure absolute paths to
ghostscript and texlive executables.
[inputs]: Add ghostscript.
---
 gnu/packages/emacs-xyz.scm | 19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 3482686ccd..e5d748b611 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -175,6 +175,7 @@
   #:use-module (gnu packages xdisorg)
   #:use-module (gnu packages shells)
   #:use-module (gnu packages sqlite)
+  #:use-module (gnu packages ghostscript)
   #:use-module (gnu packages gnupg)
   #:use-module (gnu packages video)
   #:use-module (gnu packages haskell-xyz)
@@ -1914,6 +1915,21 @@ as a library for other Emacs packages.")
        #:exclude '("^tests/" "^latex/README")
        #:phases
        (modify-phases %standard-phases
+         (add-after 'unpack 'configure
+           (lambda* (#:key inputs #:allow-other-keys)
+             (emacs-substitute-variables "preview.el"
+               ("preview-gs-command"
+                (string-append (assoc-ref inputs "ghostscript") "/bin/gs")))
+             (substitute* "preview.el"
+               (("\"dvipng ")
+                (string-append "\"" (assoc-ref inputs "texlive")
+                               "/bin/dvipng "))
+               (("\"dvips ")
+                (string-append "\"" (assoc-ref inputs "texlive")
+                               "/bin/dvips "))
+               (("\"pdf2dsc ")
+                (string-append "\"" (assoc-ref inputs "ghostscript")
+                               "/bin/pdf2dsc ")))))
          (add-after 'install 'install-doc
            (lambda* (#:key outputs #:allow-other-keys)
              (let* ((out (assoc-ref outputs "out"))
@@ -1928,7 +1944,8 @@ as a library for other Emacs packages.")
     (native-inputs
      `(("perl" ,perl)))
     (inputs
-     `(("texlive" ,(texlive-union (list texlive-amsfonts)))))
+     `(("ghostscript" ,ghostscript)
+       ("texlive" ,(texlive-union (list texlive-amsfonts)))))
     (home-page "https://www.gnu.org/software/auctex/")
     (synopsis "Integrated environment for TeX")
     (description
-- 
2.28.0

[bug#43185] [PATCH] gnu: emacs-auctex: Configure executable paths.

[Mathieu Othacehe]

Hello Arun,

> * gnu/packages/emacs-xyz.scm (emacs-auctex): Configure absolute paths to
> ghostscript and texlive executables.
> [inputs]: Add ghostscript.

This looks fine to me.

Thanks,

Mathieu

bug#43185: [PATCH] gnu: emacs-auctex: Configure executable paths.

[Arun Isaac]

Pushed to master, thanks for the review!