[bug#53851] [PATCH] gnu: Add emacs-latex-preview-pane.

[bug#53851] [PATCH] gnu: Add emacs-latex-preview-pane.

[Jai Vetrivelan]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: 0001-gnu-Add-emacs-latex-preview-pane.patch --]
[-- Type: text/x-patch, Size: 2176 bytes --]

From 470a2605d4adf98e77aee9171e4292e9e7c41561 Mon Sep 17 00:00:00 2001
From: Jai Vetrivelan <jaivetrivelan@gmail.com>
Date: Mon, 7 Feb 2022 21:44:36 +0530
Subject: [PATCH] gnu: Add emacs-latex-preview-pane.

* gnu/packages/emacs-xyz.scm (emacs-latex-preview-pane): new variable.
---
 gnu/packages/emacs-xyz.scm | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index bec613e381..5251e503c9 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -19018,6 +19018,34 @@ (define-public emacs-lacarte
     (description "Execute menu items as commands, with completion.")
     (license license:gpl3)))
 
+(define-public emacs-latex-preview-pane
+  (let ((commit "5297668a89996b50b2b62f99cba01cc544dbed2e")
+        (revision "0"))
+      (package
+       (name "emacs-latex-preview-pane")
+       (version (git-version "20151021" revision commit))
+       (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://github.com/jsinglet/latex-preview-pane")
+                      (commit commit)))
+                (file-name (git-file-name name version))
+                (sha256
+                 (base32
+                  "1m4f5p53275k8i9p0y105kkrp9nx1bwn6726my9s5dwnjhr5dnp2"))))
+       (build-system emacs-build-system)
+       (arguments
+        ;; The welcome and error message templates are in .txt format.
+        '(#:include '("\\.el$" "\\.txt$")))
+       (home-page "https://www.emacswiki.org/emacs/LaTeXPreviewPane")
+       (synopsis "Makes LaTeX editing less painful by providing a updatable
+preview pane")
+       (description "@code{latex-preview-pane} is a minor mode for Emacs that
+enables you to preview your LaTeX files directly in Emacs. It supports PDF
+previews, your choice of pdflatex or xelatex, and it highlights errors in your
+LaTeX buffer.")
+       (license license:gpl3+))))
+
 (define-public emacs-isearch-prop
   (let ((commit "4a2765f835dd115d472142da05215c4c748809f4")
         (revision "2"))

base-commit: 4396fef04717eebb061b2c808087ca638f87a826
-- 
2.34.0

[bug#53851] [PATCH] gnu: Add emacs-latex-preview-pane.

[Nicolas Goaziou]

Hello,

Jai Vetrivelan <jaivetrivelan@gmail.com> writes:

> Subject: [PATCH] gnu: Add emacs-latex-preview-pane.

Thank you.
>
> * gnu/packages/emacs-xyz.scm (emacs-latex-preview-pane): new variable.

Nitpick: capitalize "New".

> +       (arguments
> +        ;; The welcome and error message templates are in .txt format.
> +        '(#:include '("\\.el$" "\\.txt$")))

You can switch to G-expressions:

    (arguments
     (list
      #:include #~(list "\\.el$" "\\.txt$")))

This is a bit more verbose, tho.

> +       (home-page "https://www.emacswiki.org/emacs/LaTeXPreviewPane")

Why is the home-page pointing to EmacsWiki? Shouldn't it point to the
repository instead?

> +       (synopsis "Makes LaTeX editing less painful by providing a updatable
> +preview pane")

I suggest something shorter and more neutral:

    "Preview pane to ease LaTeX editing in Emacs"

> +       (description "@code{latex-preview-pane} is a minor mode for Emacs that
> +enables you to preview your LaTeX files directly in Emacs. It supports PDF
> +previews, your choice of pdflatex or xelatex, and it highlights errors in your
> +LaTeX buffer.")

Nitpick: I would remove all "your" references…

  "@code{latex-preview-pane} is a minor mode for previewing LaTeX files
  directly in Emacs. It supports PDF previews, with either
  @command{pdflatex} or @command{xelatex}, and highlights errors in the
  LaTeX buffer.

Could you send an updated patch?

Regards,
-- 
Nicolas Goaziou

[bug#53851] [PATCH] gnu: Add emacs-latex-preview-pane.

[Jai Vetrivelan]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: 0001-gnu-Add-emacs-latex-preview-pane.patch --]
[-- Type: text/x-patch, Size: 2204 bytes --]

From c543f0911bd3498213e72fb631353a8f58925c9c Mon Sep 17 00:00:00 2001
From: Jai Vetrivelan <jaivetrivelan@gmail.com>
Date: Tue, 8 Feb 2022 06:14:46 +0530
Subject: [PATCH] gnu: Add emacs-latex-preview-pane.

* gnu/packages/emacs-xyz.scm (emacs-latex-preview-pane): New variable.
---
 gnu/packages/emacs-xyz.scm | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 679c50845b..50cd6dee74 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -19018,6 +19018,34 @@ (define-public emacs-lacarte
     (description "Execute menu items as commands, with completion.")
     (license license:gpl3)))
 
+(define-public emacs-latex-preview-pane
+  (let ((commit "5297668a89996b50b2b62f99cba01cc544dbed2e")
+        (revision "0"))
+    (package
+      (name "emacs-latex-preview-pane")
+      (version (git-version "20151021" revision commit))
+      (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://github.com/jsinglet/latex-preview-pane")
+                      (commit commit)))
+                (file-name (git-file-name name version))
+                (sha256n
+                 (base32
+                  "1m4f5p53275k8i9p0y105kkrp9nx1bwn6726my9s5dwnjhr5dnp2"))))
+      (build-system emacs-build-system)
+      (arguments
+       (list
+        ;; The welcome and error message templates are in .txt format.
+        #:include #~(list "\\.el$" "\\.txt$")))
+      (home-page "https://github.com/jsinglet/latex-preview-pane")
+      (synopsis "Preview pane to ease LaTeX editing in Emacs")
+      (description "@code{latex-preview-pane} is a minor mode for previewing
+LaTeX files directly in Emacs.  It supports PDF previews, with either
+@command{pdflatex} or @command{xelatex}, and highlights errors in the LaTeX
+buffer.")
+      (license license:gpl3+))))
+
 (define-public emacs-isearch-prop
   (let ((commit "4a2765f835dd115d472142da05215c4c748809f4")
         (revision "2"))

base-commit: 64c599c9a33e83d64609a8791fcd8685fc1b9b61
prerequisite-patch-id: 019ac4d3b1d8438a3018d10851691e62fce6f1f4
-- 
2.34.0

bug#53851: [PATCH] gnu: Add emacs-latex-preview-pane.

[Nicolas Goaziou]

Hello,

Jai Vetrivelan <jaivetrivelan@gmail.com> writes:

> Subject: [PATCH] gnu: Add emacs-latex-preview-pane.

Applied. Thank you.

Regards,
-- 
Nicolas Goaziou